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

Add language parameter where missing in catalog side #11019

Merged
merged 2 commits into from Jun 6, 2022
Merged

Add language parameter where missing in catalog side #11019

merged 2 commits into from Jun 6, 2022

Conversation

nmpetkov
Copy link
Contributor

@nmpetkov nmpetkov commented Jun 5, 2022

Thi resolves the issue #10887

Description: Since in Opencart 4 the selected language in the front part of the site is transmitted as a GET parameter, for all links and ajax calls that depend on the language, they must contain the selected language as a parameter. This is done in general, in this PR are added in places where it is missing and this leads to unexpected changes in the language for the user.

@condor2
Copy link
Contributor

condor2 commented Jun 5, 2022

On register.twig you write url and index with UPPERCASE :)

@condor2
Copy link
Contributor

condor2 commented Jun 5, 2022

If you are still here :), add alert-dismissible on

$('#alert').prepend('<div class="alert alert-success">

shipping.twig

@nmpetkov
Copy link
Contributor Author

nmpetkov commented Jun 5, 2022

On register.twig you write url and index with UPPERCASE :)

For some reason a technical error from using the shortcut ... corrected!

@condor2
Copy link
Contributor

condor2 commented Jun 5, 2022

@nmpetkov don't forget this.

If you are still here :), add alert-dismissible on

$('#alert').prepend('<div class="alert alert-success">

shipping.twig

@danielkerr danielkerr merged commit 368b816 into opencart:master Jun 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants