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

batches: add settings for org level batch changes admin #50604

Merged
merged 1 commit into from
Apr 28, 2023

Conversation

BolajiOlajide
Copy link
Contributor

Part 1 #50447

This PR adds a new settings field for org admins to specify if all members of the Org are Batch Changes admin. With this, org-members who are not site administrators can execute/apply/run batch changes in the org namespace.

Test plan

Just a settings field addition. A quick test is to open the settings editor and enter orgs.allMembersBatchChangesAdmin in the editor, it should be a valid field.

CleanShot 2023-04-13 at 17 57 18@2x

@BolajiOlajide BolajiOlajide added the batch-changes Issues related to Batch Changes label Apr 13, 2023
@BolajiOlajide BolajiOlajide requested a review from a team April 13, 2023 16:58
@BolajiOlajide BolajiOlajide self-assigned this Apr 13, 2023
@cla-bot cla-bot bot added the cla-signed label Apr 13, 2023
@BolajiOlajide BolajiOlajide reopened this Apr 14, 2023
schema/settings.schema.json Show resolved Hide resolved
schema/schema.go Show resolved Hide resolved
@BolajiOlajide BolajiOlajide force-pushed the bo/allow-org-level-batch-changes-admin-setting branch from 313e6cf to 1e81c60 Compare April 28, 2023 13:49
@BolajiOlajide BolajiOlajide merged commit ce79296 into main Apr 28, 2023
@BolajiOlajide BolajiOlajide deleted the bo/allow-org-level-batch-changes-admin-setting branch April 28, 2023 14:13
@courier-new
Copy link
Contributor

Ha, not that it matters much, but I think you committed my suggestions on GitHub but then rebased and force-pushed from your local branch without pulling them first, so they got overwritten. 😜

github-actions bot pushed a commit that referenced this pull request May 30, 2023
keegancsmith pushed a commit that referenced this pull request May 31, 2023
#52656)

Part 1 #50447

This PR adds a new settings field for org admins to specify if all
members of the Org are Batch Changes admin. With this, org-members who
are not site administrators can execute/apply/run batch changes in the
org namespace.

Backport ce79296 from #50604

Co-authored-by: Bolaji Olajide <25608335+BolajiOlajide@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
batch-changes Issues related to Batch Changes cla-signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants