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

Actor group not nullable #805

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Actor group not nullable #805

wants to merge 1 commit into from

Conversation

Erikvv
Copy link
Member

@Erikvv Erikvv commented Jun 23, 2023

Add database constraint that an Actor always has a group. This constraint holds true for the Actors currently in the acceptance environment.

Cost-benefit module is simpler if this constraint is satisfied.

@Erikvv Erikvv force-pushed the actor-group-not-null branch from 5048a87 to 9861bc8 Compare June 23, 2023 15:22
Add database constraint that an Actor always has a group. This
constraint holds true for the Actors currently in the acceptance
environment.

Cost-benefit module is simpler if this constraint is satisfied.
@Erikvv Erikvv force-pushed the actor-group-not-null branch from 9861bc8 to b39dcc8 Compare June 23, 2023 15:23
@Erikvv
Copy link
Member Author

Erikvv commented Jun 26, 2023

Gaat nog fout in test setup of fixtures.

@Erikvv Erikvv marked this pull request as draft June 26, 2023 17:19
@Erikvv Erikvv force-pushed the main branch 25 times, most recently from 52781e5 to 113aa8b Compare August 28, 2023 09:58
@Erikvv Erikvv force-pushed the main branch 3 times, most recently from 1d21775 to 379e1d7 Compare September 5, 2023 13:13
@Erikvv Erikvv force-pushed the main branch 6 times, most recently from 45b15b2 to 403229b Compare January 13, 2024 17:36
@Erikvv Erikvv force-pushed the main branch 2 times, most recently from dd48ce8 to fe92f7d Compare April 8, 2024 15:49
@Erikvv Erikvv force-pushed the main branch 3 times, most recently from a15e7ea to f8b551c Compare May 22, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant