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-26279)[PRO] refactor: cleanup Offerer and Venue components #9862

Merged
merged 8 commits into from
Dec 28, 2023

Conversation

AlbericTrancart
Copy link
Contributor

@AlbericTrancart AlbericTrancart commented Dec 28, 2023

But de la pull request

Refacto iso fonctionnelle qui déplace et simplifie des props dans le but de pouvoir réutiliser plus facilement pour la feature
La plupart des props déplacés n'avait pas de raison d'exister (vs juste descendre la venue ou l'offerer) et pour les valeurs calculées et passées aux enfants le coût en perf est anecdotique (très peu d'AC ont plus de 5 lieux) de répéter le calcul dans les composants enfants.

Tests possibles :

  • vérifier les lieux de la sandbox qui ont des étapes en cours (ajout de rib, etc)
  • vérifier l'affichage des lieux numériques ("offres numériques") vs physiques
  • le lien "ajouter un lieu" quand il n'y a que des offres numériques

Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-26279

Vérifications

  • J'ai écrit les tests nécessaires
  • 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

Copy link
Contributor

github-actions bot commented Dec 28, 2023

Visit the preview URL for this PR (updated for commit 30c2516):

https://pc-pro-testing--pr9862-pc-26279-5k45gwyt.web.app

(expires Sat, 30 Dec 2023 16:10:14 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1

Copy link
Contributor

@smokhtari-passculture smokhtari-passculture left a comment

Choose a reason for hiding this comment

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

ça fait du bien de plus voir autant de props

hasNonFreeOffer,
isFirstVenue,
}: VenueProps) => {
export const Venue = ({ offerer, venue, isFirstVenue }: VenueProps) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

🔥

@AlbericTrancart AlbericTrancart merged commit 26c4362 into master Dec 28, 2023
5 of 6 checks passed
@AlbericTrancart AlbericTrancart deleted the pc-26279 branch December 28, 2023 16:42
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.

2 participants