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

(PC-31316)[PRO] feat: enable multiprovider set-up in venue settings #13883

Merged

Conversation

tcoudray-pass
Copy link
Contributor

@tcoudray-pass tcoudray-pass commented Aug 29, 2024

But de la pull request

Ticket Jira : https://passculture.atlassian.net/browse/PC-31316

Dans ce ticket :

  • Ajout du paramètre dateCreated dans le serializer VenueProviderResponse
  • Ajout d'un nouveau composant VenueProviderCard en remplacement du plus rudimentaire VenueProviderItem (le figma cible)
  • Mise en place de la logique pour permettre de configurer plusieurs provider pour une venue.

image
image

Vérifications

  • J'ai écrit les tests nécessaires
  • J'ai mis à jour le fichier des plans de tests du portail pro si nécessaire
  • J'ai mis à jour la liste des routes et des titres de pages du portail pro si j'en ai rajouté/modifié ou supprimé une.
  • J'ai relu attentivement les migrations, en particulier pour éviter les locks, et je préviens les équipes Shérif et Data
  • J'ai ajouté des screenshots pour d'éventuels changements graphiques

@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/PC-31316-new-venue-provider-card branch from df60039 to 3bc98f5 Compare August 29, 2024 15:00
Copy link
Contributor

github-actions bot commented Aug 29, 2024

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

@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/PC-31316-new-venue-provider-card branch 3 times, most recently from f92c383 to a3a8693 Compare August 29, 2024 16:02
@tcoudray-pass tcoudray-pass changed the title (PC-31316)[API] feat: add dateCreated in VenueProvider serializer (PC-31316)[PRO] feat: enable multiprovider set-up in venue settings Aug 29, 2024
@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/PC-31316-new-venue-provider-card branch 8 times, most recently from 4b32771 to a8257aa Compare September 2, 2024 08:07
Copy link
Contributor

@mageoffray mageoffray left a comment

Choose a reason for hiding this comment

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

@xdekoninck-pass
Copy link
Contributor

La PR semble casser les tests E2E

@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/PC-31316-new-venue-provider-card branch from a8257aa to b6cb4d5 Compare September 2, 2024 13:40
@tcoudray-pass tcoudray-pass reopened this Sep 2, 2024
@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/PC-31316-new-venue-provider-card branch from b6cb4d5 to f0c8158 Compare September 2, 2024 14:05
@tcoudray-pass
Copy link
Contributor Author

La PR semble casser les tests E2E

Je crois que ce n'était pas dû à ma PR. J'ai fait un rebase avec master et les tests sont passés.

@tcoudray-pass tcoudray-pass merged commit bfa92c2 into master Sep 2, 2024
26 checks passed
@tcoudray-pass tcoudray-pass deleted the tcoudray-pass/PC-31316-new-venue-provider-card branch September 2, 2024 14:38
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.

4 participants