ETQ admin je suis informé des infos remontées par le champ SIRET #11013
Conversation
colinux
force-pushed
the
admin-knows-api-fields
branch
from
282ecfb to
2bea6ed
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #11013 +/- ##
==========================================
- Coverage 84.40% 84.39% -0.02%
==========================================
Files 1181 1181
Lines 26026 26039 +13
Branches 4907 4909 +2
==========================================
+ Hits 21968 21975 +7
- Misses 4058 4064 +6 ☔ View full report in Codecov by Sentry. |
colinux
force-pushed
the
admin-knows-api-fields
branch
2 times, most recently
from
3d46004 to
eecb4eb
Compare
colinux
force-pushed
the
admin-knows-api-fields
branch
from
eecb4eb to
f42d8de
Compare
colinux
force-pushed
the
admin-knows-api-fields
branch
4 times, most recently
from
ef85336 to
f6fd8bc
Compare
colinux
force-pushed
the
admin-knows-api-fields
branch
3 times, most recently
from
ccba83f to
ed423f3
Compare
colinux
force-pushed
the
admin-knows-api-fields
branch
from
ed423f3 to
49791e8
Compare
colinux
changed the title
colinux
force-pushed
the
admin-knows-api-fields
branch
from
49791e8 to
8bc3017
Compare
colinux
force-pushed
the
admin-knows-api-fields
branch
from
8bc3017 to
3b67345
Compare
colinux
force-pushed
the
admin-knows-api-fields
branch
from
3b67345 to
75945a9
Compare
| @@ -422,6 +422,26 @@ def administrateurs | |||
| @admins = paginate(@admins, 'users.email') | |||
| end | |||
|
|
|||
| def api_champ_columns | |||
| _, @type_de_champ = @procedure.draft_revision.coordinate_and_tdc(params[:stable_id]) | |||
| regex_prefix = /^#{Regexp.escape(@type_de_champ.libelle)}([^\p{L}]+SIRET)?[^\p{L}]+/ | |||
There was a problem hiding this comment.
je trouve ça dommage de sortir de la grosse artillerie alors qu'on a l'information.
contre proposition
- on ajoute les attributs
prefixetnameaux colonnes - on définit
def label = prefix + name
et tu fais un simple filter_map { _1.name }
There was a problem hiding this comment.
Au départ j'étais parti sur un truc dans ce genre là. Ça amenait bcp de code "juste pour ça" ici surtout que c'est plus qu'une histoire de préfixe. Je sais plus bien mais ya déjà une notion de prefix et label qui a une logique un peu différente, ça menait de la confusion et exposait de nouvelles méthodes publiques. Avec je sais plus qui on était pas chaud pour changer le contrat maintenant.
Par contre sur le moyen terme quand on aura déployé ça à autre tdc , le bloc personne morale etc, et qu'on y verra plus clair sur nos besoins, on pourra carrément un plan plus uniforme
123d363
Closes #11008 en vue du webinaire du 12/12
Seconde partie de #11099
Quelques spécificités tant que toutes les données associées au SIRET ne sont pas correctement gérées en tant que colonnes (certaines données établissement, d'adresse, générées pour les exports etc…)