The profile links in Masthead should be customizable
See original GitHub issueDetailed description
Describe in detail the issue you’re having. Currently Masthead uses Translation service to get profile menu items and urls. According to the source code, it retrieves the config json at https://www.ibm.com/common/v18/js/data/jsononly/uken.json
In that json file, unfortunately the login url is still the old one: https://idaas.iam.ibm.com/idaas/mtfim/sps/authsvc?PolicyId=urn:ibm:security:authentication:asf:basicldapuser&Target={{window.location}}
For the new w3id with openId, it should be https://idaas.iam.ibm.com/idaas/oidc/endpoint/default/authorize?response_type=token&client_id=v18loginprod&state={{window.location}}&redirect_uri=https://myibm.ibm.com/OIDCHandler.html&scope=openid&nonce=8675309
Is this a feature request (new component, new icon), a bug, or a general issue? a bug. If the profile links prop can be customizable, then it could be a feature.
Is this issue related to a specific component? Masthead
What did you expect to happen? What happened instead? What would you like to see changed? I’ve tried to set up another translation host using the env variable
process.env.TRANSLATION_HOST
, but without success.
What browser are you working in? Chrome
What version of the IBM.com Library are you using? 1.10.1
What offering/product do you work on? Any pressing ship or release dates we should be aware of? Storage Digital Platform
Steps to reproduce the issue
- Step one: open https://www.ibm.com/it-infrastructure/storage/flash/offers/uk-en
- Step two: select the “Log in” menu item in the profile menu
Additional information
- This issue is a duplicate of the issue (#4118) opened by @shrinandhini2801. Issue 4118 is consolidated into this issue and closed.
- Notes
Issue Analytics
- State:
- Created 3 years ago
- Comments:23 (10 by maintainers)
Hi @motou ! We actually just made a change in the data source to point to the
login.ibm.com
endpoint this morning. You may need to refresh your browser to see the changes.However, we are pushing up a fix in
1.10.2
that will dynamically set thewindow.location
, which is currently available for testing inv1.10.2-rc.2
.cc: @ljcarot @annawen1 @kennylam @larahanlon2
I have scheduled the meeting tomorrow morning at 10:00 ET. It looks like Kenny has finished development. I have included him in the meeting.
@kennylam