-
Notifications
You must be signed in to change notification settings - Fork 204
Update docs for Google SSO #5044
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
Conversation
Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5044 +/- ##
==========================================
+ Coverage 22.45% 22.47% +0.01%
==========================================
Files 522 522
Lines 57041 57041
==========================================
+ Hits 12809 12818 +9
+ Misses 43204 43195 -9
Partials 1028 1028 ☔ View full report in Codecov by Sentry. |
t-kikuc
left a comment
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.
We should update here too:
| | Google SSO | Incubating | |
Signed-off-by: Yoshiki Fujikane <[email protected]>
|
Thank you! removed in 544c9bb |
t-kikuc
left a comment
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.
Thank you!
khanhtc1202
left a comment
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 👍
What this PR does / why we need it:
Google SSO is not yet supported, but some documents describe it.
It's confusing, so I fixed them.
This is because we previously planned the feature but didn't Implement it.
Which issue(s) this PR fixes:
Part of #1161
Does this PR introduce a user-facing change?: