1. 19 Jun, 2019 4 commits
    • Patrick Derichs's avatar
      Simplify result of find_noteable · 5469d21d
      Patrick Derichs authored
      5469d21d
    • Patrick Derichs's avatar
      Remove unneeded parentheses · 9079085f
      Patrick Derichs authored
      9079085f
    • Patrick Derichs's avatar
      b5b56588
    • Patrick Derichs's avatar
      Use NotesFinder to fetch notes on API and Controllers · 932a9a0c
      Patrick Derichs authored
      Fix missing iid query on NotesFinder
      
      Changed parameters of find_noteable, 
      so changes across a few files were needed.
      MergeRequest also requires iid instead of id query
      
      Make NotesFinder fail with RecordNotFound again
      
      Add specs for target_iid
      
      Using RSpec tablesyntax for target_iid specs
      
      Revert "Using RSpec tablesyntax for target_iid specs"
      
      This reverts commit ba45c7f569a.
      
      Allow find_by! here
      
      Fix variable name
      
      Add readable check
      
      Revert "Add readable check"
      
      This reverts commit 9e3a1a7aa39.
      
      Remove unnecessary assignment
      
      Add required changes for EE
      
      Fix parameter count
      
      Reduce code duplication by extracting a noteable module method
      
      The call to find_noteable was redundant so
      multiple files and lines have changed in that
      commit to use the newly introduced module
      method `noteable`.
      
      Replace casecmp with include check
      
      Add parent_type parameter
      
      
      Revert "Reduce code duplication by extracting
      a noteable module method"
      
      This reverts commit 8c0923babff16.
      
      Method is no longer needed
      
      Check whether noteable can be read by user
      932a9a0c
  2. 18 Jun, 2019 36 commits