-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Migrate old metadata table to the new ones #41465
Conversation
ce0b337
to
96bbc20
Compare
Pinging nick just for DB expertise |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing autoloader update
96bbc20
to
794595e
Compare
1df1bbf
to
28c15df
Compare
Signed-off-by: Louis Chemineau <louis@chmn.me>
28c15df
to
95938d3
Compare
It seems we have a chicken&egg issue here when upgrading via web UI: The page throws an internal server error, caused by the missing
Ah least it works via Btw, is it intended that there is now both:
|
I think a similar issue has already been fixed. @ArtificialOwl maybe when could add another check?
FYI, the logs are not caused by this PR. |
It's of course a bit weird that |
Please raise a new issue, add the feedback 28 label and ping AndyScherzinger |
Signed-off-by: Louis Chemineau louis@chmn.me