Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

Extend endpoint registration UI to support Client ID and Client Secret #2920

Merged
merged 6 commits into from
Sep 11, 2018

Conversation

nwmac
Copy link
Contributor

@nwmac nwmac commented Aug 28, 2018

This PR adds the ability to specify the Client ID and Client Secret to use when registering an endpoint.

The back-end changes are already in place for this, so this PR adds the necessary fields into the UI

@nwmac nwmac self-assigned this Aug 28, 2018
@cfdreddbot
Copy link

Hey nwmac!

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
Copy link

codecov bot commented Aug 28, 2018

Codecov Report

Merging #2920 into v2-master will decrease coverage by 0.01%.
The diff coverage is 19.56%.

@@              Coverage Diff              @@
##           v2-master    #2920      +/-   ##
=============================================
- Coverage      46.73%   46.72%   -0.02%     
=============================================
  Files            607      607              
  Lines          26169    26180      +11     
  Branches        5926     5930       +4     
=============================================
+ Hits           12230    12232       +2     
- Misses         13939    13948       +9

Copy link
Contributor

@richard-cox richard-cox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, can merge once gates pass

- Show redirect_uri message underneath sso checkbox
- Merge with new optional client id/secret section split these two components
- Text for redirect_uri is now limited by form, alernative is to place SSO under optional section
@richard-cox richard-cox merged commit 569aa16 into v2-master Sep 11, 2018
@richard-cox richard-cox deleted the client-params-ui branch September 11, 2018 15:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants