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

Refine redmine_hearts:migrate_from_vote_on_issues task (for master) #52

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

nishidayuya
Copy link
Contributor

@nishidayuya nishidayuya commented Nov 5, 2024

#51 is precedes this pull-request. This pull-request's changes are e0d2189 and 786b05f .

CI result is here: https://github.com/nishidayuya/redmine_hearts/actions/runs/11682722316

About e0d2189

Sometimes, vote_on_issues table has no issue_id records. Because vote_on_issues.issue_id does not have NOT NULL constraint (NULL-able). vote_on_issues.issue_id ditto.

So, I want to skip invalid records when migrating from vote_on_issues. I create this commit.

About 786b05f

My environment has vote_on_issues records by locked users. Because I want to migrate them, I create this commit.

I think another solution that is skipping migration for records by locked users. But I want to do data migration as many as possible.

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.

2 participants