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

1124 viewers are not allowed in rotations #1577

Merged
merged 6 commits into from
Mar 20, 2023

Conversation

Ukochka
Copy link
Contributor

@Ukochka Ukochka commented Mar 20, 2023

What this PR does

Added additional filtering for roles 0 and 1 for rodations. Only Editors and Admins are allowed to be in rotation

Which issue(s) this PR fixes

#1124

@Ukochka Ukochka requested a review from a team March 20, 2023 11:35
@Ukochka Ukochka added the pr:no public docs Added to a PR that does not require public documentation updates label Mar 20, 2023
CHANGELOG.md Outdated
@@ -15,6 +15,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Changed

- Updated wording in some Slack messages to use 'Alert Group' instead of 'Incident' ([1565](https://github.com/grafana/oncall/pull/1565))
- Filtering for Editors/Admins was added to rotation form. It is not allowed to assign Viewer to rotation ([1577]<https://github.com/grafana/oncall/pull/1577>)
Copy link
Member

Choose a reason for hiding this comment

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

This should point to the GitHub issue not the PR itself. It should be #1124

@Ukochka Ukochka added this pull request to the merge queue Mar 20, 2023
@Ukochka Ukochka merged commit 046d1dc into dev Mar 20, 2023
@Ukochka Ukochka deleted the 1124-viewers-are-not-allowed-in-rotations branch March 20, 2023 14:09
brojd pushed a commit that referenced this pull request Sep 18, 2024
# What this PR does
Added additional filtering for roles 0 and 1 for rodations. Only Editors
and Admins are allowed to be in rotation

## Which issue(s) this PR fixes
#1124
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:no public docs Added to a PR that does not require public documentation updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants