Use NotesFinder to fetch notes on API and Controllers
There are some places across the app like api/notes.rb
and app/controllers/concerns/issuable_actions.rb
that are not using
the NotesFinder
to fetch notes. We should update it to use the finder in all places.
We should take care about not to decrease performance when doing so.
The following discussion from !19193 (merged) should be addressed:
-
@felipe_artur started a discussion: (+1 comment) We should open ~"technical debt" to use
NotesFinder
here. Not sure if we still don't for performance reasons.
Edited by Felipe Artur