BigW Consortium Gitlab

  1. 16 Aug, 2016 1 commit
    • Fix failing tests relating to backporting ee!581. · dd3b738d
      Timothy Andrew authored
      1. `GitPushService` was still using `{merge,push}_access_level_attributes` instead
         of `{merge,push}_access_levels_attributes`.
      
      2. The branches API creates access levels regardless of the state of the
         `developers_can_{push,merge}` parameters. This is in line with the UI,
         where Master access is the default for a new protected branch.
      
      3. Use `after(:build)` to create access levels in the
         `protected_branches` factory, so that `factories_spec` passes. It
         only builds records, so we need to create access levels on `build` as
         well.
  2. 29 Jul, 2016 2 commits
    • Use `Gitlab::Access` to protected branch access levels. · 0a8aeb46
      Timothy Andrew authored
      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.
    • Have the `branches` API work with the new protected branches data model. · 01d190a8
      Timothy Andrew authored
      1. The new data model moves from `developers_can_{push,merge}` to
         `allowed_to_{push,merge}`.
      
      2. The API interface has not been changed. It still accepts
         `developers_can_push` and `developers_can_merge` as options. These
         attributes are inferred from the new data model.
      
      3. Modify the protected branch create/update services to translate from
         the API interface to our current data model.
  3. 19 Jul, 2016 4 commits
  4. 01 Jul, 2016 1 commit
  5. 30 Jun, 2016 1 commit
  6. 06 Apr, 2016 1 commit
    • Changed the argument of not_found for 'unprotect' · d023a852
      Yasser Hussain authored
      not_found appends string "Not Found" to the argument causing the resulting message to be "Branch does not exist Not Found" which is an incorrect error message.
      
      Changed the argument of not_found! for 'unprotect'  command to "Branch" from "Branch does not exist".  This makes the final error message to appear as "Branch Not Found" which is correct and same as error messages for other commands like 'protect'.
  7. 31 Mar, 2015 1 commit
  8. 25 Mar, 2015 1 commit
  9. 24 Mar, 2015 1 commit
    • Unescape branch param to delete · 862e1e6f
      Dan Tudor authored
      Branch names that contain `/` return a 405 error when being deleted because the slash is escaped to `%2F`
      This patch will unescape the param prior to  executing the delete action.
  10. 30 Dec, 2014 1 commit
  11. 30 Oct, 2014 1 commit
  12. 25 Sep, 2014 1 commit
  13. 21 Sep, 2014 1 commit
  14. 03 Sep, 2014 1 commit
  15. 23 May, 2014 1 commit
  16. 22 May, 2014 1 commit
  17. 08 May, 2014 1 commit
  18. 01 Apr, 2014 1 commit
  19. 31 Mar, 2014 1 commit