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
Enable symfony layout feature flag by default #35180
Enable symfony layout feature flag by default #35180
Conversation
Hi, thanks for this contribution! I found some issues with the Pull Request description:
Would you mind having a look at it? This will help us understand how interesting your contribution is, thank you very much! About linked issuesPlease consider opening an issue before submitting a Pull Request:
(Note: this is an automated message, but answering it will reach a real human) |
d36cc76
to
24ce66d
Compare
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.
There are some tiny issues:
Language code is encoded:
var full_language_code = 'en\u002Dus';
var full_cldr_language_code = 'en\u002Dus';
changeFormLanguageUrl
missing, is it needed?
…rom the Symfony Security service
I just checked By the way |
@jolelievre I digged in and the feature is used. If you go to Advanced parameters >Employees and enable "Memorize the language used in Admin panel forms", it will enable So we fix the encoding and then it's ready. :-) |
No QA for this one even though it enables the new symfony layout on the branch This will allow us to getearly feedback And a full manual QA is planned after we finish upgrading the remaining dependencies (removal of sensio framework extra bundle and upgrade of ApiPlatform) |
Symfony layout: https://github.com/jolelievre/ga.tests.ui.pr/actions/runs/7670270819
Legacy layout: https://github.com/jolelievre/ga.tests.ui.pr/actions/runs/7670297902