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

Update templates to get translations using new API #18423

Merged
merged 29 commits into from
May 3, 2022

Conversation

LalaRzayeva
Copy link
Contributor

Fixes #15964


Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

@CLAassistant
Copy link

CLAassistant commented Apr 20, 2022

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@mshima mshima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reduce a fell generated code differences.

Copy link
Member

@mshima mshima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

_getClientTranslation template support should resolve many problems.
The template support must be applied to _getEntityClientTranslation too.

@LalaRzayeva
Copy link
Contributor Author

@mshima I removed the TODO comments from the translation, which you seem to have corrected, so there are only 6 comments left that require your attention. Maybe you could fix them yourself as it's still not clear to me what to do with them.

mshima
mshima previously approved these changes Apr 30, 2022
@mshima mshima marked this pull request as ready for review April 30, 2022 01:32
@mshima mshima closed this Apr 30, 2022
@mshima mshima reopened this Apr 30, 2022
@mshima
Copy link
Member

mshima commented May 1, 2022

No clue why vue is failing coverage test.

@mshima mshima merged commit 3ecc4f6 into jhipster:main May 3, 2022
@mshima
Copy link
Member

mshima commented May 3, 2022

@LalaRzayeva thanks for your contribution, don't forget to claim the bounty.

@mshima
Copy link
Member

mshima commented May 3, 2022

@DanielFran @pascalgrimaud this is a cool feature to be highlighted into release notes.

@LalaRzayeva
Copy link
Contributor Author

@pascalgrimaud
Copy link
Member

@LalaRzayeva : approved

@LalaRzayeva LalaRzayeva deleted the issue-15964 branch May 8, 2022 10:36
@DanielFran DanielFran added this to the 7.9.0 milestone Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New application without internationalization is always in English
5 participants