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 for user defined themes #5682

Merged
merged 3 commits into from Jan 12, 2019

Conversation

8 participants
@adelowo
Copy link
Member

adelowo commented Jan 9, 2019

Fixes #5681

Friendly ping @wyattoday @kimpenhaus

@adelowo adelowo changed the title fix migration fix migration for user defined themes Jan 9, 2019

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Jan 9, 2019

Codecov Report

❗️ No coverage uploaded for pull request base (master@2b36bdd). Click here to learn what that means.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #5682   +/-   ##
=========================================
  Coverage          ?   37.75%           
=========================================
  Files             ?      323           
  Lines             ?    47596           
  Branches          ?        0           
=========================================
  Hits              ?    17972           
  Misses            ?    27034           
  Partials          ?     2590
Impacted Files Coverage Δ
models/migrations/v77.go 0% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b36bdd...9f3cda2. Read the comment docs.

@bkcsoft bkcsoft added the lgtm/need 2 label Jan 9, 2019

@techknowlogick techknowlogick added this to the 1.8.0 milestone Jan 10, 2019

@bkcsoft bkcsoft added lgtm/need 1 and removed lgtm/need 2 labels Jan 10, 2019

@allenh1

This comment has been minimized.

Copy link

allenh1 commented Jan 11, 2019

Is there a way I can work around this temporarily (while this is getting merged)?

@kimpenhaus

This comment has been minimized.

Copy link

kimpenhaus commented Jan 11, 2019

You can try and make an update to the SQL table user and set theme to empty string (or gitea or arc-green).

@bkcsoft bkcsoft added lgtm/done and removed lgtm/need 1 labels Jan 12, 2019

lunny and others added some commits Jan 12, 2019

@techknowlogick techknowlogick merged commit 9e9d1b8 into go-gitea:master Jan 12, 2019

2 checks passed

approvals/lgtm this commit looks good
continuous-integration/drone/pr the build was successful
Details

@zeripath zeripath referenced this pull request Jan 13, 2019

Open

Permission denied after latest docker upgrade #5688

2 of 2 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment