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-32205)[API] chore: Migrate bookshops using Titelive to new API for stock update #14478

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tcoudray-pass
Copy link
Contributor

@tcoudray-pass tcoudray-pass commented Oct 4, 2024

But de la pull request

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

Cette PR ne sera pas mergée mais runnée via le job console.

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 changed the title (PC-31580)[API] chore: (PC-32205) Migrate bookshops using Titelive to new API for stock update (PC-32205)[API] chore: Migrate bookshops using Titelive to new API for stock update Oct 4, 2024
Copy link
Contributor

github-actions bot commented Oct 4, 2024

mypy cop report: 450 (master) ↗ 453 (your branch)

@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/PC-32205-Titetlive-migration branch from 7e00b1b to 7b4a066 Compare October 7, 2024 06:55
@jeremieb-pass jeremieb-pass force-pushed the tcoudray-pass/PC-32205-Titetlive-migration branch from c21690b to 0d863af Compare October 10, 2024 09:51
@jeremieb-pass
Copy link
Contributor

J'ai ajouté (puis retiré) un commit pour permettre d'exécuter la session de 10h jusqu'à 13h suite à un soucis au niveau de l'infra.

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