-
Notifications
You must be signed in to change notification settings - Fork 98
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
feat(container): updated user widget and support dynamic updates #13412
base: master
Are you sure you want to change the base?
Conversation
{user.legalform === LEGAL_FORMS.INDIVIDUAL ? | ||
`${user.firstname} ${user.name}`: user.organisation } |
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.
From the ticket I would have implemented this way:
{user.legalform === LEGAL_FORMS.INDIVIDUAL ? | |
`${user.firstname} ${user.name}`: user.organisation } | |
{user.legalform === LEGAL_FORMS.CORPORATION? | |
user.organisation : `${user.firstname} ${user.name}` } |
>{user.legalform === LEGAL_FORMS.INDIVIDUAL ? | ||
`${user.firstname} ${user.name}` : user.organisation}</span> |
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.
Same as #13412 (comment)
import { LANGUAGES } from '@ovh-ux/manager-config'; | ||
import { getShellClient } from '../../../../shell'; |
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.
Can you try and inject the shell client (shellClient
) directly in the constructor (you should then be able to use it this way: shellClient.getShellClient()
)
It should be exposed in packages/manager/apps/dedicated/client/app/app.module.js line 236:
.constant('shellClient', shellClient)
ref:MANAGER-14829 Signed-off-by: Omar ALKABOUSS MOUSSANA <[email protected]>
c2ebfb7
to
ae4ae52
Compare
Quality Gate passedIssues Measures |
ref:MANAGER-14829
master
Only FR translations have been updatedBreaking change is mentioned in relevant commitsDescription
Related