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-31894)[API] feat: remove description double writing #14209

Merged

Conversation

mgeoffray-pass
Copy link
Contributor

But de la pull request

Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-31894
Do not save description in offer when a product is linked to this offer

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

@mgeoffray-pass mgeoffray-pass changed the base branch from master to mageoffray/pc-29599-bis-delete-description September 18, 2024 09:02
@mgeoffray-pass mgeoffray-pass force-pushed the mageoffray/PC-31894-remove-description-double-writing branch from ba7085f to 174df6a Compare September 18, 2024 12:07
@@ -287,10 +289,15 @@ def test_build_new_offers_from_stock_details(self):
provider_id=provider.id,
)

# We need to commit to have the offer description property working
Copy link
Contributor

Choose a reason for hiding this comment

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

Question : même pas avec un flush() ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui, avec un flush j'ai offer.description = None

api/tests/core/providers/test_api.py Outdated Show resolved Hide resolved
Base automatically changed from mageoffray/pc-29599-bis-delete-description to master September 19, 2024 08:38
@mgeoffray-pass mgeoffray-pass force-pushed the mageoffray/PC-31894-remove-description-double-writing branch 2 times, most recently from a4615d8 to 34ef587 Compare September 19, 2024 08:41
@mgeoffray-pass mgeoffray-pass force-pushed the mageoffray/PC-31894-remove-description-double-writing branch from 34ef587 to 87579b3 Compare September 19, 2024 08:47
Copy link
Contributor

mypy cop report: 450 (master) ↘ 449 (your branch)

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

@mgeoffray-pass mgeoffray-pass merged commit 17de507 into master Sep 19, 2024
22 of 23 checks passed
@mgeoffray-pass mgeoffray-pass deleted the mageoffray/PC-31894-remove-description-double-writing branch September 19, 2024 09:05
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