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

Manage getAdminBaseLink in LegacyContext::getAdminLink #10326

Merged
merged 1 commit into from Sep 7, 2018

Conversation

Projects
None yet
6 participants
@jolelievre
Contributor

jolelievre commented Sep 6, 2018

Questions Answers
Branch? develop
Description? Manage multi store urls in LegacyContext, this prevents being redirecting from one shop domain to another
Type? bug fix
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #10043
How to test? More details in the issue (requires your shop to have different domains) #10043

This change is Reviewable

@marionf marionf added QA ✔️ and removed waiting for QA labels Sep 7, 2018

@matks matks merged commit 1f9007d into PrestaShop:develop Sep 7, 2018

2 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@marionf marionf added this to the 1.7.5.0 milestone Sep 7, 2018

@matks

This comment has been minimized.

Show comment
Hide comment
@matks

matks Sep 7, 2018

Contributor

Thank you @jolelievre

Contributor

matks commented Sep 7, 2018

Thank you @jolelievre

@jolelievre jolelievre deleted the jolelievre:multistore-legacycontext branch Sep 7, 2018

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