Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add decision for (dis)approving minutes #343

Open
tomudding opened this issue Sep 25, 2023 · 2 comments
Open

Add decision for (dis)approving minutes #343

tomudding opened this issue Sep 25, 2023 · 2 comments

Comments

@tomudding
Copy link
Member

tomudding commented Sep 25, 2023

What would you like?

Decision types for approving and disapproving meeting minutes.

Why is this needed?

To prevent the board from making mistakes.

Other information

No response

@rinkp
Copy link
Member

rinkp commented Sep 25, 2023

Perhaps include the "thanks to the minute taker" and "with mentioned changes"

I don't think we should include a version (strangely this has been done some times)

@tomudding
Copy link
Member Author

I have mentioned this in the past but never added it to this issue. With the introduction of this new SubDecision we will have 3 SubDecisions with the approval and changes properties. I strongly believe we should move those to a trait (otherwise we will get another level of class inheritance).

As Rink mentioned we should probably not add a version property, so I also think we do not need to abstract other properties of these SubDecisions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants