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

Fix migration from pre-v15 to 1.2.0 #2460

Merged

Conversation

lafriks
Copy link
Member

@lafriks lafriks commented Sep 4, 2017

Otherwise when upgrading from either gogs or pre migration v15 it will fail in later steps when selecting from users table with error that user table does not have column keep_email_private

Fixes #2206 and #2437

@lafriks lafriks added this to the 1.2.0 milestone Sep 4, 2017
@lafriks lafriks mentioned this pull request Sep 4, 2017
@lafriks lafriks force-pushed the fix_migration_from_old_versions branch from 943891f to 56c31bf Compare September 4, 2017 10:33
@lunny
Copy link
Member

lunny commented Sep 4, 2017

LGTM

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Sep 4, 2017
@sapk
Copy link
Member

sapk commented Sep 4, 2017

LGTM

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Sep 4, 2017
@lunny lunny merged commit a569d3d into go-gitea:master Sep 5, 2017
lafriks added a commit to lafriks-fork/gitea that referenced this pull request Sep 5, 2017
@lunny lunny added the backport/done All backports for this PR have been created label Sep 5, 2017
@lafriks lafriks deleted the fix_migration_from_old_versions branch September 13, 2017 06:13
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrations bug
4 participants