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-32511)[PRO] Ajout de donnée sur la sandbox pour gérer tous les cas de displayedStatus #14662

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rprasquier-pass
Copy link
Contributor

But de la pull request

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

L'ensemble des cas se trouve sur le document : https://docs.google.com/spreadsheets/d/1yQvw2AA_PxlDtHRmFbB6ORmuSfCoN5-HXVd5sIn465g/edit?pli=1&gid=0#gid=0

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

Copy link
Contributor

@jcicurel-pass jcicurel-pass left a comment

Choose a reason for hiding this comment

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

Good 👌

Le seul truc qui m'embête c'est les CANCELLED, où on mélange les auto-cancel et des cancel manuels, mais on ne voit pas la différence en regardant seulement le statut des objets

Aussi si tu as besoin de regarder la raison d'annulation il faudra le préciser sur ces objets (à faire dans un second temps si besoin peut être)

Copy link
Contributor Author

@rprasquier-pass rprasquier-pass left a comment

Choose a reason for hiding this comment

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

pourquoi reimbursementPoint dans le name ?

C'est un des lieux utilisé pour le remboursement

Copy link
Contributor Author

@rprasquier-pass rprasquier-pass left a comment

Choose a reason for hiding this comment

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

détail : je l'appellerais pas corner_cases parce qu'il y a tous les cas dedans, _different_status_list ou qq chose comme ça ?

Il n'y a pas tout les cas, mais je prends le point

@rprasquier-pass rprasquier-pass force-pushed the PC-32511-eac-back-ajouter-des-offres-et-reservations-dans-la-sandbox-pour-pouvoir-tester-les-differents-cas-de-displayed-status branch from f33fe64 to dafa132 Compare October 21, 2024 10:02
@rprasquier-pass rprasquier-pass marked this pull request as draft October 21, 2024 10:22
@rprasquier-pass rprasquier-pass marked this pull request as ready for review October 21, 2024 10:22
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