Add use cases/label for Liens Capitalistiques #1818
Conversation
skelz0r
force-pushed
the
label/liens-capitalistiques
branch
from
eba4a48 to
2c9c356
Compare
skelz0r
changed the title
There was a problem hiding this comment.
PR Overview
This PR introduces a new use case/label for "Liens Capitalistiques" to support issues API-4337 and API-4340.
- Updates the French locale files with new entries for "dgfip/liens_capitalistiques" and its corresponding label.
- Adjusts the view templates to conditionally render use cases and optional use cases based on their presence.
Reviewed Changes
| File | Description |
|---|---|
| config/locales/api_entreprise/cas_usages_entries.fr.yml | Added new "dgfip/liens_capitalistiques" entries to the list of use cases. |
| config/locales/api_entreprise/fr.yml | Added a new label "DGFIP |
| app/views/api_particulier/endpoints/_use_cases.html.erb | Updated conditional rendering to check for non-empty use cases using .any?. |
| app/views/api_entreprise/endpoints/_use_cases.html.erb | Expanded conditional logic to handle both use_cases and use_cases_optional. |
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
|
Commits pour dorine 9cee988 et 2c9c356 (ref: https://mattermost.incubateur.net/betagouv/pl/9i7im4qme7gidkkmpyhbte3r6h ) Commits pour Thomas: les 2 autres (purement technique) |
|
Le label est related etalab/data_pass#719 |
There was a problem hiding this comment.
Cas d'usage commande publique :
-
Est-ce utile lors de la phase d'instruction ? Pourquoi ? Référence législative au code de CP ?
-
Est-ce utile pour le suivi prestataires ? Si oui, ref législative
Aides publiques :
(je suppose que c'est que pour de l'instruction et pas du pré-remplissage) quel utilité ?
Subventions asso | Portail GRU (les deux cas d'usages) :
l'API n'est pas indiquée comme injustifiée => Je suppose ici qu'il faut mettre une croix rouge comme pour les liasses ?
J'ai pas voulu bloquer donc j'ai mis un comment mais pour moi cette PR est rouge d'un point de vue métier
|
@Charlottecho --^ |
Do not display blocks if there is no use cases Hide optional block is not present
DorineLam
force-pushed
the
label/liens-capitalistiques
branch
from
2c9c356 to
8d5b1ad
Compare
|
J'ai rouvert une PR car je n'avais pas fini : #1826 |
|
AH, j'ai pas cherché en voyant que c'était validé |
|
pas de problème |
Closes https://linear.app/pole-api/issue/API-4337
Closes https://linear.app/pole-api/issue/API-4340