Skip to content
This repository has been archived by the owner on Dec 26, 2022. It is now read-only.

Pouvoir limiter à des intervalles de dates #68

Open
bobot opened this issue Jun 1, 2021 · 6 comments
Open

Pouvoir limiter à des intervalles de dates #68

bobot opened this issue Jun 1, 2021 · 6 comments

Comments

@bobot
Copy link

bobot commented Jun 1, 2021

Permettre de configurer un interval de date pour la première et (possiblement deuxième injection) serait utile pour que l'on puisse éliminer automatiquement les possibilités qui tombe pendant ces vacances.

La date est déjà parsé donc le test est aisé. Mais je bloque plus sur comment définir une interface simple pour choisir des intervalles de dates possible. Un widget déjà existant?

@dunglas
Copy link
Owner

dunglas commented Jun 1, 2021

C'est une idée d'évolution que j'avais en tête, mais je n'ai pas eu le temps de m'y coller.

On pourrait rajouter cette option dans une page d'options de l'extension : https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Options_pages

Concernant l'implémentation, a mon avis on devrait utiliser le format standard de règles de récurrence RRULE définit dans la RFC iCalendar (RFC 5545). @thcolin avait fait une interface compatible avec ce format dans le module Sylius que l'on avait sorti pendant le premier confinement pour permettre au commerçant de faire du click and collect. Le code doit pouvoir s'adapter relativement aisément : https://github.com/coopTilleuls/CoopTilleulsSyliusClickNCollectPlugin/blob/main/src/Resources/views/Admin/Form/_scripts.html.twig

@pirquessa
Copy link
Collaborator

On pourrait rajouter cette option dans une page d'option de l'extension : https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Options_pages

Ca me semble bien trop obscur pour l'utilisateur moyen d'accéder à cette page de configuration.

@dunglas
Copy link
Owner

dunglas commented Jun 1, 2021

@pirquessa c'est la manière standard de faire... on ne va pas changer ça. On peut toujours rajouter un lien depuis la popup qui ouvre cette page.

@pirquessa
Copy link
Collaborator

Avec un lien pourquoi pas. Mais meme comme ca, ca me semble un peu étrange d'agir sur les centres depuis la popup et d'agir sur les horaires sur une autre page.

@dunglas
Copy link
Owner

dunglas commented Jun 1, 2021

À voir oui. Il y a déjà beaucoup de trucs dans la popup. Peut-être qu'on devrait juste y laisser le bouton on/off et les logs, et déplacer le reste dans la page d'options (avec un bouton dans la popup) comme fait Privacy Badger ?
Après faut voir ce que ça donne, si ça ne prend pas trop de place ça peut sûrement rester dans la popup oui.

@pirquessa
Copy link
Collaborator

pirquessa commented Jun 1, 2021

Un systèmes de tabs ?

  1. UI des centres
  2. Options
  3. Logs

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants