-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Conversation
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.
Reduce a fell generated code differences.
...r/src/main/webapp/app/account/password-reset/finish/password-reset-finish.component.html.ejs
Outdated
Show resolved
Hide resolved
...r/src/main/webapp/app/account/password-reset/finish/password-reset-finish.component.html.ejs
Outdated
Show resolved
Hide resolved
...r/src/main/webapp/app/admin/user-management/detail/user-management-detail.component.html.ejs
Outdated
Show resolved
Hide resolved
...rs/client/templates/angular/src/main/webapp/app/account/settings/settings.component.html.ejs
Outdated
Show resolved
Hide resolved
...rs/client/templates/angular/src/main/webapp/app/account/sessions/sessions.component.html.ejs
Outdated
Show resolved
Hide resolved
...gular/src/main/webapp/app/account/password-reset/init/password-reset-init.component.html.ejs
Outdated
Show resolved
Hide resolved
...rs/client/templates/angular/src/main/webapp/app/account/password/password.component.html.ejs
Outdated
Show resolved
Hide resolved
...rs/client/templates/angular/src/main/webapp/app/account/password/password.component.html.ejs
Outdated
Show resolved
Hide resolved
...rs/client/templates/angular/src/main/webapp/app/account/register/register.component.html.ejs
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.
_getClientTranslation
template support should resolve many problems.
The template support must be applied to _getEntityClientTranslation
too.
generators/client/templates/angular/src/main/webapp/app/admin/health/health.component.html.ejs
Outdated
Show resolved
Hide resolved
generators/client/templates/angular/src/main/webapp/app/home/home.component.html.ejs
Outdated
Show resolved
Hide resolved
3e6645c
to
31f237a
Compare
...gular/src/main/webapp/app/entities/delete/entity-management-delete-dialog.component.html.ejs
Outdated
Show resolved
Hide resolved
...ates/angular/src/main/webapp/app/entities/detail/entity-management-detail.component.html.ejs
Outdated
Show resolved
Hide resolved
...ates/angular/src/main/webapp/app/entities/detail/entity-management-detail.component.html.ejs
Outdated
Show resolved
Hide resolved
...ent/templates/angular/src/main/webapp/app/entities/list/entity-management.component.html.ejs
Outdated
Show resolved
Hide resolved
generators/client/templates/angular/src/main/webapp/app/admin/health/health.component.html.ejs
Outdated
Show resolved
Hide resolved
...ates/angular/src/main/webapp/app/entities/update/entity-management-update.component.html.ejs
Outdated
Show resolved
Hide resolved
...s/client/templates/angular/src/main/webapp/app/layouts/profiles/page-ribbon.component.ts.ejs
Outdated
Show resolved
Hide resolved
...ent/templates/angular/src/main/webapp/app/entities/list/entity-management.component.html.ejs
Outdated
Show resolved
Hide resolved
...ates/angular/src/main/webapp/app/entities/detail/entity-management-detail.component.html.ejs
Outdated
Show resolved
Hide resolved
@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. |
generators/client/templates/angular/src/main/webapp/app/admin/health/health.component.html.ejs
Outdated
Show resolved
Hide resolved
...s/client/templates/angular/src/main/webapp/app/layouts/profiles/page-ribbon.component.ts.ejs
Outdated
Show resolved
Hide resolved
...ates/angular/src/main/webapp/app/entities/update/entity-management-update.component.html.ejs
Outdated
Show resolved
Hide resolved
...ent/templates/angular/src/main/webapp/app/entities/list/entity-management.component.html.ejs
Outdated
Show resolved
Hide resolved
...ates/angular/src/main/webapp/app/entities/detail/entity-management-detail.component.html.ejs
Outdated
Show resolved
Hide resolved
...ates/angular/src/main/webapp/app/entities/update/entity-management-update.component.html.ejs
Outdated
Show resolved
Hide resolved
No clue why vue is failing coverage test. |
@LalaRzayeva thanks for your contribution, don't forget to claim the bounty. |
@DanielFran @pascalgrimaud this is a cool feature to be highlighted into release notes. |
Bug bounty claimed at https://opencollective.com/generator-jhipster/expenses/75300 |
@LalaRzayeva : approved |
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.