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

Allow unowned notification policies #83

Merged
merged 1 commit into from
Oct 29, 2024
Merged

Allow unowned notification policies #83

merged 1 commit into from
Oct 29, 2024

Conversation

bbarnes52
Copy link
Contributor

@bbarnes52 bbarnes52 commented Oct 25, 2024

Allows creation of unowned notification policies and in-place updates to remove an owner from an existing policy.

Scenario test build 24 completed successfully.

@bbarnes52 bbarnes52 requested a review from a team as a code owner October 25, 2024 18:03
@bbarnes52 bbarnes52 marked this pull request as draft October 25, 2024 18:03
@bbarnes52 bbarnes52 force-pushed the bgb/unownedpolicies branch 3 times, most recently from 2f83edb to 7749721 Compare October 25, 2024 21:03
@bbarnes52 bbarnes52 force-pushed the bgb/unownedpolicies branch from 7749721 to 71de783 Compare October 28, 2024 17:45
@bbarnes52 bbarnes52 marked this pull request as ready for review October 28, 2024 21:27
@bbarnes52 bbarnes52 merged commit bcdf75d into main Oct 29, 2024
4 checks passed
@bbarnes52 bbarnes52 deleted the bgb/unownedpolicies branch October 29, 2024 15:27
bbarnes52 added a commit that referenced this pull request Oct 29, 2024
Slug should not be allowed for inline policies. This invariant was
inadvertently relaxed in #83 though never released.

Scenario test build 25 completed successfully.
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.

2 participants