Loading
The source project of this merge request has been removed.
Fix viewing default push rules on a Geo secondary
What does this MR do?
Removes a database update in a GET operation, allowing the page to work in a Geo secondary
Are there points in the code the reviewer needs to double check?
Why was this MR needed?
Fixes a 500 error visiting /admin/push_rules on a secondary.
Screenshots (if relevant)
Does this MR meet the acceptance criteria?
- Changelog entry added, if necessary
- Tests added for this feature/bug
- Review
- Has been reviewed by Backend
- Conform by the merge request performance guides
- Conform by the style guides
- Squashed related commits together
- Internationalization required/considered
- If paid feature, have we considered GitLab.com plan and how it works for groups and is there a design for promoting it to users who aren't on the correct plan
What are the relevant issue numbers?
Closes #4110 (closed)
Edited by Toon Claes