-
-
Notifications
You must be signed in to change notification settings - Fork 13
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(console): my auth url #2640
Conversation
1418257
to
4e9dd1e
Compare
@picassobanana could you tell me what you think about this? |
4e9dd1e
to
f78c0a7
Compare
a2130aa
to
81fa118
Compare
Looking good! @szkl We are just missing an apostrophe there in "won’t" and if possible I would also add text-overflow: ellipsis; if the link gets larger than the container. |
Thanks. The typo is fixed. I'll add the ellipsis. |
81fa118
to
76c0117
Compare
|
||
const authorizationScope = appDetails.app?.scopes | ||
? Array.from(appDetails.app.scopes) | ||
: ['openid', 'profile'] |
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.
Remove 'profile' from defaults array.
76c0117
to
c0b198e
Compare
Tested with the following two mentions: |
Disable if not published |
Is the domain used before in any environment? |
Domain yeah, subdomain no. I didn't investigate what went wrong, probably not related to PR. |
No behavior change in the passport. Do you mean disable/hide the auth URL? |
c0b198e
to
95fddc0
Compare
95fddc0
to
ce27027
Compare
Description
Adds a read only field to test authorization page for the user application.
Related Issues
Testing
Checklist