-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Authentication Plugin API fixes #8725
Conversation
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.
@RomanNikitenko |
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.
It is expected, to make the GitHub Pull Requests plugin work you need to authenticate it separately, see item 9 of the How to test section. |
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.
Thanks for the explanation @vinokurig
Everything works as described.
What it does
sign out
menu if a session from given authentication provider is already exists in the authentication menu.How to test
"@theia/git": "^1.5.0"
fromexamples/browser/package.json
file.plugins
folder.plugins
folder.plugins
folder.plugins
folder.Publish to GitHub
command and allow the authentication session:sign out
item:Yes
and check the accounts menu, it must reset to the initial state:Review checklist
Reminder for reviewers