BigW Consortium Gitlab
1. It makes sense to reuse these constants since we had them duplicated in the previous enum implementation. This also simplifies our `check_access` implementation, because we can use `project.team.max_member_access` directly. 2. Use `accepts_nested_attributes_for` to create push/merge access levels. This was a bit fiddly to set up, but this simplifies our code by quite a large amount. We can even get rid of `ProtectedBranches::BaseService`. 3. Move API handling back into the API (previously in `ProtectedBranches::BaseService#translate_api_params`. 4. The protected branch services now return a `ProtectedBranch` rather than `true/false`. 5. Run `load_protected_branches` on-demand in the `create` action, to prevent it being called unneccessarily. 6. "Masters" is pre-selected as the default option for "Allowed to Push" and "Allowed to Merge". 7. These changes were based on a review from @rymai in !5081.
Name |
Last commit
|
Last update |
---|---|---|
.. | ||
api.rb | Loading commit data... | |
api_guard.rb | Loading commit data... | |
award_emoji.rb | Loading commit data... | |
branches.rb | Loading commit data... | |
builds.rb | Loading commit data... | |
commit_statuses.rb | Loading commit data... | |
commits.rb | Loading commit data... | |
deploy_keys.rb | Loading commit data... | |
entities.rb | Loading commit data... | |
files.rb | Loading commit data... | |
group_members.rb | Loading commit data... | |
groups.rb | Loading commit data... | |
helpers.rb | Loading commit data... | |
internal.rb | Loading commit data... | |
issues.rb | Loading commit data... | |
keys.rb | Loading commit data... | |
labels.rb | Loading commit data... | |
license_templates.rb | Loading commit data... | |
merge_requests.rb | Loading commit data... | |
milestones.rb | Loading commit data... | |
namespaces.rb | Loading commit data... | |
notes.rb | Loading commit data... | |
project_hooks.rb | Loading commit data... | |
project_members.rb | Loading commit data... | |
project_snippets.rb | Loading commit data... | |
projects.rb | Loading commit data... | |
repositories.rb | Loading commit data... | |
runners.rb | Loading commit data... | |
services.rb | Loading commit data... | |
session.rb | Loading commit data... | |
settings.rb | Loading commit data... | |
sidekiq_metrics.rb | Loading commit data... | |
subscriptions.rb | Loading commit data... | |
system_hooks.rb | Loading commit data... | |
tags.rb | Loading commit data... | |
templates.rb | Loading commit data... | |
todos.rb | Loading commit data... | |
triggers.rb | Loading commit data... | |
users.rb | Loading commit data... | |
variables.rb | Loading commit data... |