vaccin.click icon indicating copy to clipboard operation
vaccin.click copied to clipboard

feat: add max date for appointment

Open Pistashe opened this issue 2 years ago • 3 comments

L'objet de cette PR est de proposer une date max au-delà de laquelle on ne regardera plus les rdvs disponibles. Dans mon cas, l'extension me trouvait plein de rdvs mais en Mars, trop tard pour mon pass sanitaire.

C'est pas parfait, il y a encore qq faux positifs mais ça m'a permis de trouver une date de rdv dans les temps en une grosse heure.

Pistashe avatar Dec 08 '21 11:12 Pistashe

Si tu as besoin d'un coup de main avec le rebase fais moi signe, j'ai changé pas mal les mêmes fichiers que toi dans mes PR. J'aime l'idée de la date max, j'y ai déjà pensé aussi, ce serait dommage de la voir perdue.

N'hésite pas si tu as des questions.

hschaeidt avatar Dec 11 '21 09:12 hschaeidt

@dunglas : J'ai changé pour des dates relatives. Par défaut j'ai mis date d'aujd + 1 an, je me dis que ça permet d'éviter la nécessité d'ajouter une fonctionnalité de désactivation de la date max, ce qui serait un peu plus long à implémenter, qu'est-ce que t'en penses ?

@hschaeidt : Merci, je pense que le rebase est OK mais hésite pas si tu vois des pbs :) (j'ai repris tes fonctions pour la date, bcp plus propres)

Pistashe avatar Dec 12 '21 17:12 Pistashe

Ça marche pour moi.

dunglas avatar Dec 12 '21 20:12 dunglas