-
Notifications
You must be signed in to change notification settings - Fork 134
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
Support OAuth login to console - V2 #2479
Conversation
…nto shared-endpoints
…nto shared-endpoints
# Conflicts: # src/frontend/app/shared/components/list/list-types/cf-endpoints/cf-endpoint-card/endpoint-card.component.spec.ts # src/frontend/app/store/reducers/current-user-roles-reducer/current-user-role-session.reducer.ts
…org-user-permissions Disable removal of `org user` role if user has others
…ed-endpoints Allow metrics endpoint token to be shared
…unit-tests Fix unit tests
Hey andrewghobrial! Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA. |
Codecov Report
@@ Coverage Diff @@
## v2-master #2479 +/- ##
=============================================
- Coverage 70.74% 70.71% -0.03%
=============================================
Files 589 589
Lines 24802 24817 +15
Branches 5595 5599 +4
=============================================
+ Hits 17546 17550 +4
- Misses 7256 7267 +11 |
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.
LGTM - the login hook to auto-connect the CF will only work when pushed to cf, not if run locally or deployed in docker etc - I'll look to move this to a separate login hook.
@nwmac the login hook runs locally and when deployed as all in one container too. i added it to the cloudfoundry plugin not the cf-hosting plugin. |
This is v2 version of: #2139