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

Erreur 500 lors de la validation d'une réunion avec récurrence #942

Closed
frouze opened this issue Sep 11, 2023 · 2 comments · Fixed by #946
Closed

Erreur 500 lors de la validation d'une réunion avec récurrence #942

frouze opened this issue Sep 11, 2023 · 2 comments · Fixed by #946
Assignees
Labels
bug Something isn't working

Comments

@frouze
Copy link
Collaborator

frouze commented Sep 11, 2023

J'ai tenté de faire une réunion avec récurrence sous Chrome, et FF via le POD v3.3.1 de ULille et j'ai eu un souci lors de la validation de celui-ci.
J'ai tenté soit en choisissant une date de fin soit en spécifiant le nombre d'occurences et dans les deux cas j'ai droit à une erreur 500 lors de la validation finale de ma réservation.

cas 1 (date de fin) :
image
Résultat à la validation de ma réunion
image

cas 2 (nombre d'occurences) :
image
Résultat à la validation de ma réunion
image

@ptitloup
Copy link
Contributor

Bonjour Franck et merci pour cette remontée d'information.
L'erreur est due au fait de selectionnée une récurrence sur un jour qui n'est pas celui de la date de debut de la réunion.
Par exemple, nous sommes mardi et je créé une reunion pour tous les lundi.
Maintenant, comment vois-tu cette correction, faut-il selectionné le jour correspondant à celui de la date de début ou faut-il modifier la date de debut en fonction de la réccurence?
Quel est ton avis ?
Merci
Nicolas

@ptitloup ptitloup linked a pull request Sep 12, 2023 that will close this issue
3 tasks
@ptitloup ptitloup self-assigned this Sep 12, 2023
@ptitloup ptitloup added the bug Something isn't working label Sep 12, 2023
@frouze
Copy link
Collaborator Author

frouze commented Sep 12, 2023

Salut Nicolas,
Pour moi l'idée est d'ajouter les occurrences correspondants aux jours sélectionnés et qui précèdent la date de fin choisie.

Dans le cas simple :
date début = lundi, jours sélectionné = lundi et récurrence jusqu'à un mardi plus tard dans le calendrier. => on prend tous les lundis jusqu'à la veille du "mardi de fin"

dans un cas plus complexe :
date début = lundi, jours sélectionnés = lundi et jeudi avec récurrence jusqu'à un mardi => on prend tous les lundi et jeudi des semaines antérieures au mardi final plus le dernier lundi

dans un cas plus tordu
date de début un lundi, date sélectionnée un mercredi et un jeudi avec récurrence jusqu'à un mardi => prendre le lundi de départ dont on suppose qu'il est le lancement de la récurrence puis les mercredi et jeudi suivants et qui précèdent le mardi final. Ou autre alternative ne pas prendre le premier lundi considéré comme un bug mais à ce moment alerter sur le choix étrange.

Je pense que ça semble assez cohérent. mais à voir si il y a mieux.
A+
Franck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants