-
Notifications
You must be signed in to change notification settings - Fork 32
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
sub-menu(description): implemented change description #120
sub-menu(description): implemented change description #120
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
some translations missing
longDescriptionEN: description?.[0].longDescription || '', | ||
longDescriptionDE: description?.[1].longDescription || '', | ||
shortDescriptionEN: description?.[0].shortDescription || '', | ||
shortDescriptionDE: description?.[1].shortDescription || '', |
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.
better use ??
instead of ||
to assign a default value
) { | ||
return `${t( | ||
'content.apprelease.appReleaseForm.validCharactersIncludes' | ||
)} a-zA-Z0-9 !?@&#'"()[]_-+=<>/*.,;:% and should not start with @=<>*-+ #?%&_:;` |
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.
and should not start with
should also be translated
} else { | ||
return `${t( | ||
'content.apprelease.appReleaseForm.validCharactersIncludes' | ||
)} a-zA-ZÀ-ÿ0-9 !?@&#'"()[]_-+=<>/*.,;:% and should not start with @=<>*-+ #?%&_:;` |
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.
please translate words
Description
Change Description(sub menu) functionality added for active apps
Why
To enhance.
Issue
Ref: CPLP-1558,1559
Checklist
Please delete options that are not relevant.