Horaires d'ouverture : afficher un bandeau ouvert/fermé en fonction des données

[raphodn]

Quoi ?

Si les horaires d'ouverture sont remplies (et rattachées à un type "actif"), alors on est en mesure de savoir si le magasin est ouvert ou pas à l'instant t.

• on affiche l'information sous forme de bandeau
• on prend en compte les fermetures exceptionnelles
• un nouveau paramètre display_opening_hour_open_closed_header permet de cacher le bandeau si besoin

Captures d'écran

	Image
Ouvert (mobile)
Fermé (ordi)

[petitalb]

Le problème c'est les fermetures exceptionnelles ? C'est géré quelque part ? Ca serait dommage que ça soit marqué ouvert et que ça trompe les membres parce qu'on ne peut pas le gérer proprement.

[raphodn]

Le problème c'est les fermetures exceptionnelles ? C'est géré quelque part ? Ca serait dommage que ça soit marqué ouvert et que ça trompe les membres parce qu'on ne peut pas le gérer proprement.

En effet ! Je corrigerais à mon retour

[raphodn]

Je pense modifier la PR et remplacer le "badge" par un "bandeau", qui apparaîtrait sous sur le header (un peu comme les breadcrumbs actuel)

• le badge est difficilement lisible
• en cliquant sur le bandeau on pourrait faire apparaitre un popup avec les horaires d'ouverture

Le seul soucis que j'ai, c'est d'arriver à faire une requête optimisée, pour ne pas trop ralentir le chargement de chaque page. Ou alors utiliser du cache ? Ou faire le calcul (et le stocker) toutes les X minutes ?

[raphodn]

@petitalb si jamais tu veux re-regarder. j'ai fait quelques optimisations + pris en compte les fermetures exceptionnelles + remplacé le badge par un bandeau entier

[petitalb]

Je pense qu'il y aurait moyen de faire un truc un peu plus jolie en design mais je ne sais pas trop comment faire mieux. C'est top !

[raphodn]

2 limites / améliorations que je vois (la première m'a été remonté par Corentin - à l'origine de cette idée)

• par exemple si un créneau de fin de journée n'est pas pourvu. Du coup l'épicerie devra fermer plus tôt. Mais ca indiquera toujours Ouvert car ca regarde seulement les horaires d'ouverture
• ca fait un calcul un peu complexe à chaque chargement de page. il faudrait utiliser du cache ? ou stocker cette valeur dans la base de données et la mettre à jour toutes les 5 minutes ?

et je vais rajouter un paramètre pour permettre aux épiceries qui le veulent de cacher cette fonctionnalité