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.

Sign up for GitHub

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

Jump to bottom

fix(task): improuve after party backfill_dossiers_repetitions #8545

Merged
merged 1 commit into from Feb 1, 2023
Merged

fix(task): improuve after party backfill_dossiers_repetitions #8545

merged 1 commit into from Feb 1, 2023

Conversation

tchak
Copy link
Member

@tchak tchak commented Feb 1, 2023

No description provided.

@tchak tchak added the bug label Feb 1, 2023
@what-the-diff
Copy link

what-the-diff bot commented Feb 1, 2023

  • The job was updated to only create a repetition champ if it doesn't already exist
  • A new rake task is created that will run the backfill_dossier_repetition job on all dossiers with missing champs

@tchak tchak force-pushed the fix-backfill_dossiers_repetitions branch from 2987910 to 4f34533 Compare February 1, 2023 14:05
@tchak tchak force-pushed the fix-backfill_dossiers_repetitions branch from 4f34533 to dd2d3d4 Compare February 1, 2023 14:30
@tchak tchak requested a review from colinux February 1, 2023 14:57
colinux
colinux approved these changes Feb 1, 2023
View reviewed changes
Copy link
Member

@colinux colinux left a comment

Choose a reason for hiding this comment

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

j'imagine que ça passe en local ? ;)

@tchak tchak merged commit b93b8a7 into demarches-simplifiees:main Feb 1, 2023
@tchak tchak deleted the fix-backfill_dossiers_repetitions branch February 1, 2023 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@colinux colinux colinux approved these changes

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tchak @colinux