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
[Locale] Change language negotiation to RFC 4647 based #13777
Conversation
Hey Goran, thanks a lot for your contribution and welcome among our contributors :). I will try to review it ASAP (within two weeks probably). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey Goran,
thanks for the PR + pretty nice docs update. You rock. I've added a few review comments
src/Sylius/Bundle/LocaleBundle/Context/RequestHeaderBasedLocaleContext.php
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 but we should probably also mention this change in the UPGRADE file
9dbac9f
to
e002d79
Compare
Bunnyshell Preview Environment deletedAvailable commands:
|
[Locale] Prepend default channel locale when resolving by header
911bb29
to
0ceb7db
Compare
Thank you, @gseric! |
and @NoResponseMate too! |
Following this PR: #13708