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

feat(migration): add migration compatibility with states without lineage #183

Merged
merged 1 commit into from Jul 8, 2021

Conversation

hbollon
Copy link
Member

@hbollon hbollon commented Jul 2, 2021

The lineage migration will now try to recover one lineage from other states with same path or set it to null if it fail when a state don't have one.
Also fix an issue where some lineages were ignored on migration

it will try to recover one lineage from other states with same path or set it to null if it fail
* fix: some lineages ignored on migration
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 11.287% when pulling dbc9dc9 on hbollon:migrate-states-without-lineage into f632586 on camptocamp:master.

@mcanevet mcanevet requested a review from raphink July 7, 2021 07:37
@mcanevet mcanevet added this to the v2.0.0-alpha1 milestone Jul 7, 2021
@raphink
Copy link
Contributor

raphink commented Jul 8, 2021

LGTM

@raphink raphink merged commit 707f9d8 into camptocamp:master Jul 8, 2021
raphink pushed a commit that referenced this pull request Jul 8, 2021
…age (#183)

it will try to recover one lineage from other states with same path or set it to null if it fail
* fix: some lineages ignored on migration
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.

None yet

4 participants