-
Notifications
You must be signed in to change notification settings - Fork 6
removed translation feature #468
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
Conversation
Signed-off-by: bidi <bidi@apidemia.com>
Signed-off-by: bidi <bidi@apidemia.com>
Signed-off-by: bidi <bidi@apidemia.com>
alexmerlin
left a comment
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 language/translation related relics left that need to be removed:
- https://github.com/dotkernel/frontend/blob/issue-445/src/App/templates/layout/default.html.twig#L2: replace variable
{{ language_key }}with stringen - https://github.com/dotkernel/frontend/blob/issue-445/src/App/templates/layout/default.html.twig#L8: replace
V3withV5(not related to this, but should be fixed) - Delete file https://github.com/dotkernel/frontend/blob/issue-445/src/App/assets/scss/components/_language.scss
- Delete line https://github.com/dotkernel/frontend/blob/issue-445/src/App/assets/scss/index.scss#L13
- Delete line https://github.com/dotkernel/frontend/blob/issue-445/src/App/src/ConfigProvider.php#L74 then adjust the number of spaces before the arrows
- Delete line https://github.com/dotkernel/frontend/blob/issue-445/test/Unit/App/ConfigProviderTest.php#L77
Finally, rebuild app.js and app.css.
Signed-off-by: bidi <bidi@apidemia.com>
|
the changes are done - i asked for another review i must have made the js updates right as you were reviewing because they were already done |
alexmerlin
left a comment
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.
Let's try again:
- Delete file https://github.com/dotkernel/frontend/blob/issue-445/src/App/assets/scss/components/_language.scss
- Delete line https://github.com/dotkernel/frontend/blob/issue-445/src/App/assets/scss/index.scss#L13
Finally, rebuild app.js and app.css.
Signed-off-by: bidi <bidi@apidemia.com>
oops, my bad |
No description provided.