AuthorizationDefinition#editors: sort by name #280
Conversation
Better display on decision tree: easier to find entry when it is in a natural order
| @@ -21,7 +21,7 @@ def self.all | |||
| def editors | |||
| available_forms.select { |form| | |||
| form.editor.present? | |||
| }.map(&:editor).uniq(&:id) | |||
| }.map(&:editor).uniq(&:id).sort_by(&:name) | |||
There was a problem hiding this comment.
J'ai l'impression que le ticket est un petit peu plus complexe que cela.
En console, aucun problème, c'est bien listé par :name, par contre en front end, ce n'est pas le cas malheureusement.
There was a problem hiding this comment.
Sur API Entreprise y'a 2 blocs c'est pour ça (check le code). Ça améliore le display, c'est pas 100% mieux mais c'est pas grave.
Btw c'est surtout pour API P
There was a problem hiding this comment.
Sur Api-particulier ça donne ceci :
|
tbh ici c'est 100% de l'amélioration je ne comprends pas pourquoi y'a un sujet 😅 (c'est limite un scout commit) |
|
C'est surtout pour se rappeler que dans nos cas de colonne le sort n'agit pas comme nous le souhaiterions et que l'on ne doit pas clore le ticket correspondant à cette partie du code 😄 |
Better display on decision tree: easier to find entry when it is in a natural order