-
Notifications
You must be signed in to change notification settings - Fork 674
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
[FLYTE-486] Support selecting IDP based on the query parameter #4838
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.
looks good, just one nit
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #4838 +/- ##
==========================================
- Coverage 58.97% 58.96% -0.02%
==========================================
Files 644 644
Lines 55146 55160 +14
==========================================
Hits 32525 32525
- Misses 20048 20059 +11
- Partials 2573 2576 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: pmahindrakar-oss <[email protected]>
Signed-off-by: pmahindrakar-oss <[email protected]>
Signed-off-by: pmahindrakar-oss <[email protected]>
Signed-off-by: pmahindrakar-oss <[email protected]>
Signed-off-by: pmahindrakar-oss <[email protected]>
28e6f53
to
6b617c9
Compare
#61) * Added config option for IDPQuery parameter --------- Signed-off-by: pmahindrakar-oss <[email protected]>
Overview
Test Plan
Tested on internal dogfood tenant and verified without the google tab in the login screen we can login using the okta apps in the dashboard page
Rollout Plan (if applicable)
Rollout to cloud and serverless-gcp
Jira Issue
https://unionai.atlassian.net/browse/
Checklist