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

[stable25] fix(theming): fix admin images migration #36272

Merged
merged 1 commit into from
Jan 20, 2023

Conversation

skjnldsv
Copy link
Member

Fix #36206

Context

The previous job was just clearing the old images without any migration.
Now we are properly migrating them into global and then cleaning afterwards.

Before After
2023-01-20_11-36 2023-01-20_11-38

@skjnldsv skjnldsv added this to the Nextcloud 25.0.4 milestone Jan 20, 2023
@skjnldsv skjnldsv requested review from blizzz, Pytal and a team January 20, 2023 10:47
@skjnldsv skjnldsv self-assigned this Jan 20, 2023
@skjnldsv skjnldsv requested review from ArtificialOwl and icewind1991 and removed request for a team January 20, 2023 10:47
@skjnldsv skjnldsv changed the title fix(theming): fix admin images migration [stable25] fix(theming): fix admin images migration Jan 20, 2023
@skjnldsv
Copy link
Member Author

/backport to master

Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@skjnldsv
Copy link
Member Author

Btw, I'd like to go on the record and say I heavily dislike that there is no way to move a folder in appdata. Having to recreate the folder structure and create a new file for each existing one is very cumbersome

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv skjnldsv force-pushed the fix/stable25-fix-theming-admin-migration branch from 7581dbb to 2bdbd7f Compare January 20, 2023 11:09
@szaimen szaimen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 20, 2023
@szaimen szaimen merged commit 4df5983 into stable25 Jan 20, 2023
@szaimen szaimen deleted the fix/stable25-fix-theming-admin-migration branch January 20, 2023 17:44
@szaimen
Copy link
Contributor

szaimen commented Jan 20, 2023

@skjnldsv is the problem also already fixed in master?

@szaimen
Copy link
Contributor

szaimen commented Jan 20, 2023

Ah i see there is a backport to master 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug feature: theming regression
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants