8000 Fix timezone des plannings by Zarlokh · Pull Request #1791 · afup/web · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix timezone des plannings #1791

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

Merged
merged 3 commits into from
May 15, 2025
Merged

Conversation

Zarlokh
Copy link
Contributor
@Zarlokh Zarlokh commented May 14, 2025

Fix la récupération de la data planning en rajoutant la bonne timezone

@Zarlokh Zarlokh force-pushed the fix/planning-timezone branch from 5bfcb6e to 02b0863 Compare May 14, 2025 08:30
@stakovicz
Copy link
Contributor

Top !
Est-ce que c'est possible de compléter le test behat en vérifiant les heures dans la réponse d'openfeedback 😬 ?
https://github.com/afup/web/blob/master/tests/behat/features/Api/Talks/OpenfeedbackExport.feature

@Zarlokh
Copy link
Contributor Author
Zarlokh commented May 14, 2025

@stakovicz J'aimerai bien mettre une date précise pour la date de début / fin (qui se base sur la date du jour -x jours) avec une heure précise pour que ça fonctionne sauf que je vois que dans les seed, vous faites un random_int de 5 à 6 jours pour la date de début et random_int de 1 à 5 jours pour la date de fin.
Est-ce qu'il y avait une raison particulière de random_int ?
image

@stakovicz
Copy link
Contributor

@Zarlokh non je ne vois pas de raison au random. Tu peux fixer la Datetime comme tu veux.

@@ -543,4 +544,11 @@ public function theEmailShouldContainAFullUrlStartingWith(string $arg1): void
);
}
}

#[Then('the response should contains date :arg1')]
#[Then('the response should contains date :arg1 with format :arg2')]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ça ne devrait pas être plutôt "should contain" au lieu de "should contains" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bien vu, my bad 👍

@stakovicz stakovicz merged commit 2424f98 into afup:master May 15, 2025
6 checks passed
@Zarlokh Zarlokh deleted the fix/planning-timezone branch May 28, 2025 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0