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

Display date in French format #515

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Conversation

geobrun
Copy link
Contributor

@geobrun geobrun commented Mar 22, 2024

Voir #512

@camillemonchicourt
Copy link
Member

camillemonchicourt commented Mar 22, 2024

Merci. A appliquer partout, dont sur les fiches espèces ?
image

Pas certain que cela soit le cas actuellement.

@geobrun
Copy link
Contributor Author

geobrun commented Mar 22, 2024

C'est bien ce que je pensais, je n'ai pas pu vérifier partout. De notre côté, on affiche encore la carte par maille au niveau des fiches espèces, donc comme je n'ai pas cet affichage, je n'ai pas pu vérifier...

maille

La seule contrainte que je verrai, c'est si jamais l'infobulle a absolument besoin d'une chaîne de caractères et ne peut pas interpréter un format date. Dans ce cas, l'idée serait peut-être de convertir ce champ en chaîne de caractères directement dans le code qui gère la pop-up. Est-ce que tu pourrais m'indiquer où se trouve ce code stp ?

@geobrun
Copy link
Contributor Author

geobrun commented Mar 22, 2024

D'après ce que je viens de voir, c'est le fichier atlas/static/mapGenerator.js qui s'occupe de ça. Je ne sais pas si cela fonctionnera si je rajoute la fonction strftime('%d/%m/%Y') dans ce fichier Javascript, et je n'ai malheureusement pas moyen de tester ça car on affiche les infos par maille de notre côté... :(

@TheoLechemia
Copy link
Member

TheoLechemia commented Mar 22, 2024

Merci pour la contrib !
Comme le disait Xavier dans le ticket, ce serait sympa de ne pas forcer le formatage français comme on a mis en place Babel et le multilngue.
J'ai creuser un peu et flask_babel fournit un pipe pour faire exactement ce qu'on veut.
{{obs.dateobs | dateformat("short")}} -> 22/03/2024
{{obs.dateobs | dateformat("medium")}} 22 mars 2024
et ça s'adapte à la locale courante de l'utilisateur

tu veux faire la modif ?

ps : attention en mode point, il manque la "dé-stringification" de la date : https://github.com/PnX-SI/GeoNature-atlas/blob/master/atlas/modeles/repositories/vmObservationsRepository.py#L98

@geobrun
Copy link
Contributor Author

geobrun commented Mar 25, 2024

Ok, j'ai fait ce que tu m'as dit de faire Théo, je te laisse vérifier ça !

Gestion des dates en mode point
@geobrun
Copy link
Contributor Author

geobrun commented May 16, 2024

Merci. A appliquer partout, dont sur les fiches espèces ? image

Pas certain que cela soit le cas actuellement.

On vient de passer notre atlas en mode point, donc j'ai rajouté le code qui fallait pour que les dates s'affichent plus joliment, en respectant la locale de l'utilisateur comme spécifié par Théo.

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.

None yet

3 participants