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
feat(sh-admin): introducing additional SSO related server configurations to dashboard #3737
Merged
AndrewBastin
merged 13 commits into
hoppscotch:release/2024.3.0
from
JoelJacobStephen:feat/dashboard-additional-configs
Mar 8, 2024
Merged
feat(sh-admin): introducing additional SSO related server configurations to dashboard #3737
AndrewBastin
merged 13 commits into
hoppscotch:release/2024.3.0
from
JoelJacobStephen:feat/dashboard-additional-configs
Mar 8, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
JoelJacobStephen
force-pushed
the
feat/dashboard-additional-configs
branch
from
March 7, 2024 07:02
7a8ac07
to
9c21966
Compare
JoelJacobStephen
changed the title
feat: introducing additional SSO related server configurations to admin dashboard
feat(sh-admin): introducing additional SSO related server configurations to dashboard
Mar 7, 2024
JoelJacobStephen
force-pushed
the
feat/dashboard-additional-configs
branch
2 times, most recently
from
March 7, 2024 13:04
5629668
to
5acf834
Compare
JoelJacobStephen
force-pushed
the
feat/dashboard-additional-configs
branch
from
March 8, 2024 04:52
e7d089b
to
f141cd4
Compare
- Derive types for provider fields from the base `Config` type. - Unify markup.
jamesgeorge007
approved these changes
Mar 8, 2024
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.
I've pushed a few changes in 783b327, outlining the changes in the comments below along with a few pointers for future reference.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ticket
Description
This PR enables the Admin Dashboard to show additional SSO related server configurations.
The additions to SSO Configurations include:
GOOGLE_CALLBACK_URL
GOOGLE_SCOPE
GITHUB_CALLBACK_URL
GITHUB_SCOPE
MICROSOFT_CALLBACK_URL
MICROSOFT_SCOPE
MICROSOFT_TENANT
Objectives
Checks