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

Restricter: New restriction user/B #859

Merged
merged 6 commits into from
Feb 27, 2024
Merged

Conversation

ostcar
Copy link
Member

@ostcar ostcar commented Feb 26, 2024

Fixes #818

@ostcar ostcar added the feature label Feb 26, 2024
@ostcar ostcar added this to the 4.2 milestone Feb 26, 2024
@ostcar ostcar enabled auto-merge (squash) February 27, 2024 08:31
@ostcar ostcar merged commit 9eea68a into OpenSlides:main Feb 27, 2024
2 checks passed
@jsangmeister
Copy link
Contributor

@ostcar please don't merge PRs which are part of a bigger feature, see https://github.com/OpenSlides/OpenSlides/wiki/Working-with-the-meta-repository#workflow-for-changes-to-the-modelsyml. They should be merged all together to prevent inconsistencies

@ostcar
Copy link
Member Author

ostcar commented Feb 27, 2024

How can I know this?

@jsangmeister
Copy link
Contributor

In the issue, there was a linked backend issue as well as a meta issue in the main repository.

@jsangmeister
Copy link
Contributor

Also, the PR should only be merged if the meta PR gets merged right afterwards/together, since now the tests in the meta update PR are failing: #862

@ostcar ostcar deleted the issue-818 branch February 28, 2024 08:33
bastianjoel pushed a commit that referenced this pull request Apr 17, 2024
* Restricter: New restriction user/B

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

Successfully merging this pull request may close these issues.

Add new user permissions
3 participants