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-33863)[BO] feat: remove fully_sync_venue button #15792

Conversation

tcoudray-pass
Copy link
Contributor

@tcoudray-pass tcoudray-pass commented Jan 9, 2025

But de la pull request

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

Tous les providers (hors providers cinéma) étant désormais des providers qui nous poussent l'information via l'API publique, nous sommes en train de supprimer l'ancienne intégration fournisseur (où nous allions récupérer les données sur leur API).

Cette PR est une étape intermédiaire où nous supprimons la fonctionnalité qui permettait côté BO de relancer une synchronisation fournisseur.

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
  • J'ai fait la revue fonctionnelle de mon ticket

Copy link
Contributor

github-actions bot commented Jan 9, 2025

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

https://pc-pro-testing--pr15792-tcoudray-pass-pc-338-5lvdzqal.web.app

(expires Sat, 11 Jan 2025 10:13:39 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1

@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/PC-33863-remove-fullsync-function-from-BO branch from 0548e0a to fb19174 Compare January 9, 2025 09:50
Copy link
Contributor

github-actions bot commented Jan 9, 2025

mypy cop report: 465 (master) ↘ 464 (your branch)

Have yourself a merry little break and learn a few facts about the year 464.

@ataib-pass
Copy link
Contributor

je pense que tu peux supprimer fully_sync_venue_job.py aussi puisque ce n'est plus utilisé nulle part

@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/PC-33863-remove-fullsync-function-from-BO branch from fb19174 to a93f6d9 Compare January 9, 2025 10:11
@tcoudray-pass tcoudray-pass merged commit ca74254 into master Jan 9, 2025
27 of 28 checks passed
@tcoudray-pass tcoudray-pass deleted the tcoudray-pass/PC-33863-remove-fullsync-function-from-BO branch January 9, 2025 10:51
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.

3 participants