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(todoMigration): migration script for todos #587

Merged

Conversation

lepokle
Copy link
Contributor

@lepokle lepokle commented Jul 19, 2019

This commits add a migration script for changing the 'todoId' into
the 'title' for todo documents.

This is necessary due to the changes in #517.

Closes #586

Signed-off-by: Leo von Klenze [email protected]

@lepokle lepokle added this to the Release sw360-3.4.0-M1 milestone Jul 19, 2019
@lepokle lepokle added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for labels Jul 19, 2019
@maxhbr
Copy link
Contributor

maxhbr commented Jul 22, 2019

Migration script implies increases major version

This commits add a migration script for changing the 'todoId' into
the 'title' for todo documents.

This is necessary due to the changes in eclipse-sw360#517.

Closes eclipse-sw360#586

Signed-off-by: Leo von Klenze <[email protected]>
@lepokle lepokle force-pushed the lepokle/#586-add-migration-script branch from 5cdd320 to ee7b374 Compare July 22, 2019 12:39
@smrutis1
Copy link
Contributor

Tested the scripts. Its working fine.

@mcjaeger
Copy link
Contributor

hm, too late, but next time:

  • dry run
  • log output how many items have been processed ...

@mcjaeger mcjaeger merged commit 335e0f0 into eclipse-sw360:master Jul 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs code review needs general test This is general testing, meaning that there is no org specific issue to check for
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create migration script for PR #517
4 participants