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

remove is_restricted from frontend codebase #3219

Merged
merged 2 commits into from
Oct 31, 2023

Conversation

joeyorlando
Copy link
Contributor

What this PR does

The concept of AlertGroup.is_restricted is deprecated and no longer used. In a future release I will remove all references to is_restricted from the backend (doing it in separate releases to avoid any potential issues where the frontend still references it shortly after release of it being removed from the API response)

Checklist

  • Unit, integration, and e2e (if applicable) tests updated (N/A)
  • Documentation added (or pr:no public docs PR label added if not required)
  • CHANGELOG.md updated (or pr:no changelog PR label added if not required)

@joeyorlando joeyorlando added pr:no changelog pr:no public docs Added to a PR that does not require public documentation updates labels Oct 30, 2023
@joeyorlando joeyorlando requested a review from a team October 30, 2023 18:55
@joeyorlando joeyorlando merged commit 4efe1a4 into dev Oct 31, 2023
18 of 19 checks passed
@joeyorlando joeyorlando deleted the jorlando/remove-alert-group-is-restricted branch October 31, 2023 15:19
brojd pushed a commit that referenced this pull request Sep 18, 2024
# What this PR does

The concept of `AlertGroup.is_restricted` is deprecated and no longer
used. In a future release I will remove all references to
`is_restricted` from the backend (doing it in separate releases to avoid
any potential issues where the frontend still references it shortly
after release of it being removed from the API response)

## Checklist

- [ ] Unit, integration, and e2e (if applicable) tests updated (N/A)
- [ ] Documentation added (or `pr:no public docs` PR label added if not
required)
- [ ] `CHANGELOG.md` updated (or `pr:no changelog` PR label added if not
required)
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.

1 participant