Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding CNAV scopes translations #1671

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

Un3x
Copy link
Contributor

@Un3x Un3x commented Sep 19, 2024

Scope were not displayed in the BO because the translation did not exist

Scope were not displayed because the translation did not exist
@Un3x
Copy link
Contributor Author

Un3x commented Sep 19, 2024

@Samuelfaure Je sais pas trop comment ca se passe coté api entreprise. Mais j'imagine qu'il faut aussi rajouter les équivalences de ce que j'ai fais pour les nouvelles api que tu as ajoutés.

À garder en tête pour MAJ la liste des choses à faire dans le template d'intégration d'une api.

@Samuelfaure
Copy link
Contributor

@Un3x Oulah il faut faire un test pour vérifier qu'on a pas d'oublis du coup

Je m'en occupe aujourd'hui si tu veux

@skelz0r
Copy link
Member

skelz0r commented Sep 20, 2024

Une alternative est de consommer ce fichier pour lookup les noms (comme ça on garde les noms à un seul endroit)
https://raw.githubusercontent.com/etalab/data_pass/develop/config/authorization_definitions.yml

@Un3x
Copy link
Contributor Author

Un3x commented Sep 23, 2024

Oui il y a plein d'alternatives à cette solution, il me semble qu'on attendait des améliorations cotés datapass pour pouvoir faire vraiment ce que dorine projetait.

De toute facon il y a un sujet global sur les scopes.

@Un3x Un3x merged commit 1b30763 into develop Sep 23, 2024
9 checks passed
@Un3x Un3x deleted the adding-cnav-prestation-scope-translations branch September 23, 2024 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants