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

[NEW UI] Post restore page #1101

Merged
merged 3 commits into from
Jan 8, 2025

Conversation

ga-devfront
Copy link
Contributor

@ga-devfront ga-devfront commented Dec 27, 2024

Questions Answers
Description? Add post restore page
Type? new feature
BC breaks? no
Deprecations? no
Fixed ticket? N/A
Sponsor company @PrestaShopCorp
How to test? You can try to access directly with parameter &new-ui=1&route=restore-page-post-restore but without the restore task you can't access logs.

@ga-devfront ga-devfront added enhancement Blocked Status: The issue is blocked by another task labels Dec 27, 2024
@ga-devfront ga-devfront added this to the 7.0.0 milestone Dec 27, 2024
@ga-devfront ga-devfront force-pushed the feat/post-restore-page branch from 6d9754b to 73ff5a9 Compare January 7, 2025 08:06
@ga-devfront ga-devfront force-pushed the feat/post-restore-page branch from 7428c4f to 19e5a17 Compare January 8, 2025 10:10
Copy link

sonarqubecloud bot commented Jan 8, 2025

@ga-devfront ga-devfront removed the Blocked Status: The issue is blocked by another task label Jan 8, 2025
return array_merge(
$updateSteps->getStepParams($this::CURRENT_STEP),
[
'exit_link' => DIRECTORY_SEPARATOR . $this->upgradeContainer->getProperty(UpgradeContainer::PS_ADMIN_SUBDIR) . DIRECTORY_SEPARATOR . 'index.php',
Copy link
Member

Choose a reason for hiding this comment

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

As reported by @AureRita, this does not work on shops installed in a sub-folder.

Copy link
Member

Choose a reason for hiding this comment

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

I will change this on #1107

@Quetzacoalt91
Copy link
Member

Merging with a known issue that is going to be fixed on #1107

@Quetzacoalt91 Quetzacoalt91 merged commit 214adb2 into PrestaShop:dev Jan 8, 2025
37 checks passed
@ga-devfront ga-devfront deleted the feat/post-restore-page branch January 8, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants