An error occurred fetching the project authors.
  1. 02 Aug, 2016 1 commit
    • Paco Guzman's avatar
      Speedup DiffNote#active? on discussions, preloading noteables and avoid... · 8716ff7f
      Paco Guzman authored
      Speedup DiffNote#active? on discussions, preloading noteables and avoid touching git repository to return diff_refs when possible
      
      - Preloading noteable we share the same noteable instance when more than one 
      discussion refers to the same noteable.
      - Any other call to that object that is cached in that object will be for any 
      discussion.
      - In those cases where merge_request_diff has all the sha stored to build a diff_refs get that 
      diff_refs using directly those sha instead accessing to the git repository to first get the 
      commits and later the sha.
      8716ff7f
  2. 01 Aug, 2016 1 commit
    • zs's avatar
      State specific default sort order for issuables · 84a3225b
      zs authored
      Provide more sensible default sort order for issues and merge requests
      based on the following table:
      
      | type           | state  | default sort order |
      |----------------|--------|--------------------|
      | issues         | open   | last created       |
      | issues         | closed | last updated       |
      | issues         | all    | last created       |
      | merge requests | open   | last created       |
      | merge requests | merged | last updated       |
      | merge requests | closed | last updated       |
      | merge requests | all    | last created       |
      84a3225b
  3. 28 Jul, 2016 1 commit
  4. 26 Jul, 2016 2 commits
  5. 20 Jul, 2016 1 commit
  6. 19 Jul, 2016 2 commits
  7. 14 Jul, 2016 1 commit
  8. 12 Jul, 2016 2 commits
  9. 08 Jul, 2016 3 commits
  10. 07 Jul, 2016 1 commit
  11. 06 Jul, 2016 5 commits
  12. 28 Jun, 2016 1 commit
  13. 24 Jun, 2016 2 commits
    • Rémy Coutable's avatar
      Fix a wrong MR status when merge_when_build_succeeds &... · e3d464d8
      Rémy Coutable authored
      Fix a wrong MR status when merge_when_build_succeeds & project.only_allow_merge_if_build_succeeds are true
      
      The issue was that `MergeRequest#mergeable?` returns false when the CI
      state is not success and project.only_allow_merge_if_build_succeeds is
      true. In this case `Projects::MergeRequestsController#merge` would
      return the `:failed` status when enabling `merge_when_build_succeeds`,
      thus leading to a weird state and the MR never automatically merged.
      
      The fix is to disable the CI state check in the controller safeguard
      that early return the `:failed` status.
      Signed-off-by: default avatarRémy Coutable <remy@rymai.me>
      e3d464d8
    • Yorick Peterse's avatar
      Support for rendering/redacting multiple documents · d470f3d1
      Yorick Peterse authored
      This commit changes the way certain documents are rendered (currently
      only Notes) and how documents are redacted. Previously both rendering
      and redacting would run on a per document basis. The result of this was
      that for every document we'd have to run countless queries just to
      figure out if we could display a set of links or not.
      
      This commit changes things around so that redacting Markdown documents
      is no longer tied into the html-pipeline Gem. This in turn allows it to
      redact multiple documents in a single pass, thus reducing the number of
      queries needed.
      
      In turn rendering issue/merge request notes has been adjusted to take
      advantage of this new setup. Instead of rendering Markdown somewhere
      deep down in a view the Markdown is rendered and redacted in the
      controller (taking the current user and all that into account). This has
      been done in such a way that the "markdown()" helper method can still be
      used on its own.
      
      This particular commit also paves the way for caching rendered HTML on
      object level. Right now there's an accessor method Note#note_html which
      is used for setting/getting the rendered HTML. Once we cache HTML on row
      level we can simply change this field to be a column and call a "save"
      whenever needed and we're pretty much done.
      d470f3d1
  14. 23 Jun, 2016 2 commits
  15. 16 Jun, 2016 2 commits
  16. 15 Jun, 2016 2 commits
  17. 14 Jun, 2016 2 commits
  18. 07 Jun, 2016 1 commit
  19. 06 Jun, 2016 2 commits
  20. 03 Jun, 2016 4 commits
  21. 02 Jun, 2016 1 commit
  22. 01 Jun, 2016 1 commit
    • Sean McGivern's avatar
      Don't allow merges with new commits · f680eca9
      Sean McGivern authored
      Set a `sha` parameter on the MR form. If this doesn't match the HEAD of
      the source branch when the form is submitted, show a warning (like with
      a merge conflict) and don't merge the branch.
      f680eca9