cartes icon indicating copy to clipboard operation
cartes copied to clipboard

<OpenIndicator /> est faux lorsqu'un lieu est ouvert 24/24

Open kevinvennitti opened this issue 1 year ago • 3 comments

Exemple ici :

screencapture-2024-08-10-22 07 27

Dans OpeningHours.tsx, ce lieu semble avoir ìsOpen = false, ce qui conditionne la couleur de la pastille. Par contre, le message est correct car il s'appuie sur nextChange.

kevinvennitti avatar Aug 10 '24 20:08 kevinvennitti

Précisément, quand on regarde le noeud, on a opening_hours="Selon la programmation de la Cité de l'architecture". Le message Ouvert24/24 7j/7 n'est donc pas correct, il faudrait améliorer la façon dont on parse pour afficher ces messages customs qui semblent bien conformes au wiki.

waterced avatar Aug 11 '24 18:08 waterced

A parcourir le code je constate qu'on fait appel à un parser extérieur ("opening_hours": "^3.8.0",) je suppose que c'est celui-ci https://packages.ecosyste.ms/registries/npmjs.org/packages/opening_hours

waterced avatar Aug 12 '24 06:08 waterced

On a dans la doc:

Comments

  • Supports (additional) comments (Mo unknown "on appointment"; Th-Fr 09:00-18:00 open "female only"; Su closed "really")
    • The string which is delimited by double-quotes can contain any character (except a double-quote sign)
    • unknown can be omitted (just a comment (without state) will also result in unknown) - value can also be just a double-quoted string ("on appointment") which will result in unknown for any given time.

Reste à comprendre si la valeur unknown est prise en charge actuellement

waterced avatar Aug 12 '24 06:08 waterced