-
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-31316)[PRO] feat: enable multiprovider set-up in venue settings #13883
(PC-31316)[PRO] feat: enable multiprovider set-up in venue settings #13883
Conversation
df60039
to
3bc98f5
Compare
Visit the preview URL for this PR (updated for commit f0c8158): https://pc-pro-testing--pr13883-tcoudray-pass-pc-313-j03qzrcb.web.app (expires Wed, 04 Sep 2024 14:25:29 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1 |
f92c383
to
a3a8693
Compare
dateCreated
in VenueProvider serializer4b32771
to
a8257aa
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.
✅
La PR semble casser les tests E2E |
pro/src/pages/VenueSettings/VenueProvidersManager/VenueProviderList/AllocineProviderEdit.tsx
Outdated
Show resolved
Hide resolved
pro/src/pages/VenueSettings/VenueProvidersManager/VenueProviderList/CinemaProviderEdit.tsx
Outdated
Show resolved
Hide resolved
a8257aa
to
b6cb4d5
Compare
b6cb4d5
to
f0c8158
Compare
Je crois que ce n'était pas dû à ma PR. J'ai fait un rebase avec master et les tests sont passés. |
But de la pull request
Ticket Jira : https://passculture.atlassian.net/browse/PC-31316
Dans ce ticket :
dateCreated
dans le serializerVenueProviderResponse
VenueProviderCard
en remplacement du plus rudimentaireVenueProviderItem
(le figma cible)Vérifications