-
Notifications
You must be signed in to change notification settings - Fork 38
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
(PC-30417)[API] feat: new nav for retention user in sandbox #12924
Conversation
Visit the preview URL for this PR (updated for commit e7158bb): https://pc-pro-testing--pr12924-pc-30417-sandbox-ret-i0uh8mze.web.app (expires Thu, 04 Jul 2024 08:52:14 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1 |
0e89550
to
34c23c9
Compare
51fb820
to
1aa6a24
Compare
263ccbe
to
aac77ef
Compare
aac77ef
to
7184e91
Compare
(BSR)[PRO] test: fix (BSR)[PRO] test: fix (BSR)[PRO] test: new data again in sandbox comment + linter fix
7184e91
to
cfebde7
Compare
(BSR)[PRO] test: fix (BSR)[PRO] test: fix
3eadc43
to
e7158bb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il reste le user admin qui ne sera pas sur le NPP. Mais je sais pas si quelqu'un l'utilise
But de la pull request
Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-30417
Commencé par @Aliochka fini par @scolson-pass
Lors de la relecture, ne vous prenez pas trop la tête sur les données utilisées dans la sandbox si ce n'est pas parfait, ça va être remplacé rapidement....
J'ai été obligé de rajouter des contrôles intermédiaires (spinner absent, url chargée, wait, etc...) car simplement cliquer sur un bouton avec un texte et une commande testing-library ne fait pas forcément ce qu'on veut. Une instabilité
venue
est également traitée ici.Vérifications