Skip to content

Implement equal fields constraint in schema#3382

Merged
luisa-beerboom merged 39 commits intoOpenSlides:mainfrom
luisa-beerboom:3379-fix-equal-fields
Apr 16, 2026
Merged

Implement equal fields constraint in schema#3382
luisa-beerboom merged 39 commits intoOpenSlides:mainfrom
luisa-beerboom:3379-fix-equal-fields

Conversation

@luisa-beerboom
Copy link
Copy Markdown
Member

@luisa-beerboom luisa-beerboom commented Mar 4, 2026

Resolves #3379
As long as it is checked out with OpenSlides/openslides-meta#407

TODO:

  • Maybe add required fields tests
  • Reduce new equal_fields tests to a minimum

@luisa-beerboom luisa-beerboom self-assigned this Mar 4, 2026
Comment thread tests/system/action/motion/test_update.py Outdated
Copy link
Copy Markdown
Member

@hjanott hjanott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commented some parts again and also the migration test is still failing. Probably some error in the legacy example data file.

@hjanott hjanott assigned luisa-beerboom and unassigned hjanott Apr 13, 2026
@luisa-beerboom
Copy link
Copy Markdown
Member Author

Commented some parts again and also the migration test is still failing. Probably some error in the legacy example data file.

Re-running caused the error to disappear -v(·_·)v-
Maybe it was fixed via meta update, maybe it's just flaky.
Either way, I can't tell rn.

@hjanott
Copy link
Copy Markdown
Member

hjanott commented Apr 14, 2026

maybe it's just flaky.

Flaky.

@hjanott hjanott assigned luisa-beerboom and unassigned hjanott Apr 14, 2026
Copy link
Copy Markdown
Member

@hjanott hjanott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the hard work. Looks ready for me now.

@luisa-beerboom luisa-beerboom merged commit ac84607 into OpenSlides:main Apr 16, 2026
5 checks passed
luisa-beerboom added a commit to luisa-beerboom/openslides-backend that referenced this pull request Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Prevent non meeting_user from being a submitter (and probably much more)

3 participants