-
Notifications
You must be signed in to change notification settings - Fork 45
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
Improvement/2078 store user credentials in localstorage #2088
Improvement/2078 store user credentials in localstorage #2088
Conversation
Hello chengyanjin,My role is to assist you with the merge of this Status report is not available. |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list:
|
/after_pull_request=2042 |
Waiting for other pull request(s)The current pull request is locked by the after_pull_request option. In order for me to merge this pull request, run the following actions first: ➡️ Merge the
Alternatively, delete all the after_pull_request comments from this pull request. The following options are set: after_pull_request |
37b1cc8
to
51be9e8
Compare
51be9e8
to
fa0aa3f
Compare
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list:
The following options are set: after_pull_request |
fa0aa3f
to
03a282a
Compare
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.
Works great but the logout button does not work anymore.
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list:
The following reviewers are expecting changes from the author, or must review again: The following options are set: after_pull_request |
03a282a
to
edbc5bb
Compare
We will want to share the user crediential across different tabs. By default redux-oidc storge access_token in sessionStorge, in this PR we will change it to localStorage. Refs: #2078
we add the oidc-client library so it's reach out the limit of assetsize
edbc5bb
to
d5c14d6
Compare
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.
I rebuilt and tested again. Everything is ok. I cannot reproduce the logout button problem.
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list:
The following options are set: after_pull_request |
/approve |
In the queueThe changeset has received all authorizations and has been added to the The changeset will be merged in:
The following branches will NOT be impacted:
There is no action required on your side. You will be notified here once IMPORTANT Please do not attempt to modify this pull request.
If you need this pull request to be removed from the queue, please contact a The following options are set: approve, after_pull_request |
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue None. Goodbye chengyanjin. |
Component: UI, Dex
Context:
We will want to share the user credentials across different tabs so that users won't need to be authenticated again.
Summary:
We set
userStore: new WebStorageStateStore({ store: localStorage }),
by default in userManagerConfig.Acceptance criteria:
Once login, we should find the
oidc.user:https://172.21.254.12:8443/oidc:metalk8s-ui
oidc credential stored in localStorage.Closes: #2078