Skip to content

Conversation

@blizzz
Copy link
Member

@blizzz blizzz commented Oct 13, 2022

backport of #34461

I didn't take over the version bump in version.php as it comes with RC4 already anyway.

Pytal and others added 7 commits October 13, 2022 18:01
Signed-off-by: Joas Schilling <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
...and execute delete query

Signed-off-by: Arthur Schiwon <[email protected]>
- fallback to background image from old location
- migrate background images to new location as insensitive job

Signed-off-by: Arthur Schiwon <[email protected]>
@PVince81
Copy link
Member

I've tested this on master: #34461 (comment)

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.

👍

@PVince81 PVince81 enabled auto-merge October 13, 2022 17:03
@PVince81 PVince81 added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 13, 2022
$this->jobList = $jobList;
}

protected function run($argument): void {

Check notice

Code scanning / Psalm

MissingParamType

Parameter $argument has no provided type
return 'Initialize migration of background images from dashboard to theming app';
}

public function run(IOutput $output) {

Check notice

Code scanning / Psalm

MissingReturnType

Method OCA\Theming\Migration\InitBackgroundImagesMigration::run does not have a return type, expecting void
@skjnldsv skjnldsv disabled auto-merge October 13, 2022 18:17
@Pytal Pytal merged commit 7f498d0 into stable25 Oct 13, 2022
@Pytal Pytal deleted the backport/34461/stable25 branch October 13, 2022 18:26
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: dashboard feature: theming

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants