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

Remplacer les mentions visibles de M52 par M14 #23

Closed
2 tasks
thom4parisot opened this issue Feb 18, 2019 · 6 comments
Closed
2 tasks

Remplacer les mentions visibles de M52 par M14 #23

thom4parisot opened this issue Feb 18, 2019 · 6 comments

Comments

@thom4parisot
Copy link

  • dans les URLs
  • dans les textes lisibles
@thom4parisot thom4parisot added this to To do in Itération 2 (éclair) via automation Feb 18, 2019
@thom4parisot thom4parisot added this to To do in Itération 3 via automation Feb 25, 2019
@DavidBruant
Copy link
Member

J'ai l'impression que le problème à résoudre, c'est surtout faire disparaitre les mentions de M52

Parfois, la solution, c'est d'utiliser un terme plus générique ; pas forcément mettre "M14"

@thom4parisot
Copy link
Author

Tu penses à un terme générique en particulier ? Ou certains, en fonction d'usages particuliers ?

@DavidBruant
Copy link
Member

Par exemple, hierarchicalM52.js pourrait être renommé hierarchicalByFunction.js

M52Entry pourrait être remplacé par LigneBudget (ou autre chose qui fait une référence à l'élément XML du même nom, mais un peu différent parce qu'on modifie un peu les "metadata" de la ligne)

Beaucoup d'instances sont issues du fait que je découvrais le métier des finances publiques et que "M52" semblait être le qualificateur le plus pertinent

@thom4parisot
Copy link
Author

thom4parisot commented Feb 26, 2019

Dac, je vois — ça serait l'objet d'une autre issue.

Celle-ci concerne les mentions du terme M52 visibles par le grand public en parcourant la dataviz.

@DavidBruant
Copy link
Member

ah oki, j'avais mal compris
pardon pour le bruit

@ariasuni
Copy link
Collaborator

Je ne vois pas de mention M52 dans les pages que je peux visiter ni dans les CSV des labels. Je vois juste /build/m52-overview-bundle.js dans les URL à priori.

@thom4parisot thom4parisot moved this from To do to Needs review in Itération 3 Mar 1, 2019
Itération 3 automation moved this from Needs review to Done Mar 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Development

No branches or pull requests

3 participants