-
Notifications
You must be signed in to change notification settings - Fork 8
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
Affiche, appro : publier les données d'achats pour l'année en cours, avec l'option de dépublier #3936
Conversation
This reverts commit 8f2a9bc.
…nflicts and updating code to reflect new data structure of tabs
@hfroot L'idée du champ |
}, | ||
mounted() { | ||
this.publishedToggleState = this.getPublicationState(this.tab) | ||
return this.getPurchasesSummary().finally(() => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Par curiosité, pourquoi l'usage de finally
et non then
ici ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parce que une réponse prévue est un 404 (si on est dans le vue public ou il y a pas d'achats pour l'année)
Erratum : Tu l'as déjà fait sur #3942 Pour une autre PR, mais je pense que cette section n'a pas de raison de s'afficher quand on n'a pas de données (ou qu'elles sont redacted) |
Affiche, appro : gérer le cas où il y a pas de données
Closes #3910
Je suis pas convaincu.e que le champ
redacted_appro_years
est le mieux, car je pense que c'est pas "je veux dépublier les données de 2024" mais plutôt que "je veux dépublier les données en cours". Avec la solution actuelle dans cette PR, en 2025 les données en cours seront visibles à nouveau.Je propose en deuxième temps alors un nouveau champ
redact_current_year
ou qqch. Tu penses quoi ?