API: gestion des labels

[colinux]

• liste des labels du dossier
• associer un label à un dossier
• suppression d'un label à un dossier
• liste des labels de la démarche (pour récup les ids)
• ajouter des labels à une démarche: on verra plus tard si c'est demandé
• doc

[codecov]

Codecov Report

Attention: Patch coverage is 96.87500% with 2 lines in your changes missing coverage. Please review.

Project coverage is 84.60%. Comparing base (2bfd51e) to head (e1c7672).
Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
app/graphql/mutations/dossier_ajouter_label.rb	94.73%	1 Missing ⚠️
app/graphql/mutations/dossier_supprimer_label.rb	94.11%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #11201      +/-   ##
==========================================
+ Coverage   84.55%   84.60%   +0.04%     
==========================================
  Files        1196     1200       +4     
  Lines       26300    26362      +62     
  Branches     4962     4972      +10     
==========================================
+ Hits        22239    22303      +64     
+ Misses       4061     4059       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[tchak]

J'ajouterai bien un authorized? :

def self.authorized?(object, context)
    context.authorized_demarche?(object.procedure)
end

Ça ne coûte rien et ça garantit que l'objet ne sera jamais visible s'il n'est pas autorisé

[tchak]

Pourquoi expose-t-on ici DossierLabel et pas Label ? Je n'ai pas l'impression que DossierLabel contient des informations utiles. Je sais que tu m'as posé la question et je crois que j'ai mal compris. On peut en parler à l'occasion. Ce n'est pas bloquant pour merger non plus – on peut toujours changer plus tard. Le nom du type GraphQL est bon, je pense – DossierLabel.

[tchak]

Ah mais non, tu as exposé les deux sur l'API... Je voudrais qu'on en parle

[tchak]

avec authorized? sur le label, ça ne devrait pas être utile

[colinux]

j'ai rajouté le authorized sur les types, mais c'est insuffisant car si l'admin a plusieurs démarches on veut éviter qu'il associe le label d'une autre démarche

[tchak]

Suggested change

	field :labels, [Types::DossierLabelType], "Labels associés au dossier", null: false
	field :labels, [Types::LabelType], "Labels associés au dossier", null: false

[tchak]

Suggested change

	Loaders::Association.for(object.class, dossier_labels: [:label]).load(object)
	Loaders::Association.for(object.class, dossier_labels: [:label]).load(object).then { _1.map(:label) }