Skip to content

Configuration Modal Form#85

Open
javiert-okta wants to merge 7 commits intoconfiguration-modal-uifrom
configuration-modal-logic
Open

Configuration Modal Form#85
javiert-okta wants to merge 7 commits intoconfiguration-modal-uifrom
configuration-modal-logic

Conversation

@javiert-okta
Copy link
Contributor

@javiert-okta javiert-okta commented Feb 13, 2026

By submitting a PR to this repository, you agree to the terms within the Auth0 Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

The following PR adds the configuration modal component logic, which adds a form submission to let the user handle the OIDC config. The changes include:

  • Refactor debugger steps to open modal and access debugger steps and auth data
  • Add form validation and submission
  • Update parent state from modal

Screenshots

  • Dark theme

Desktop

image
  • Light Theme

Desktop

image

References

Link to Figma design

Testing

  • Verify that the configuration modal matches the design on the figma file (dark/light mode, desktop, tablet and mobile)
  • Change all possible servers (Auth0, Google, Custom)
  • Change URL config and test against it
  • Verify the state changes from the modal to the debugger component

Checklist

  • I have added documentation for new/changed functionality in this PR or in auth0.com/docs
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used, if not the default branch

@github-actions
Copy link

label="Save"
showIcon={false}
onClick={() => {
onSaveData(formValues as InitialModalData);
Copy link
Contributor

Choose a reason for hiding this comment

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

The form is not validated before saving

domain: value !== "auth0" ? SERVER_URLS[value] : "samples.auth0.com",
};
});
} catch (error) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't we show an error message instead of just silently clearing the fields?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants