-
Notifications
You must be signed in to change notification settings - Fork 1
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
MPDX-8514 + MPDX-7911 - Improve the Integrations page #1255
Conversation
Bundle sizes [mpdx-react]Compared against a4d5410 No significant changes found |
Preview branch generated at https://change-add-org-text.d3dytjb8adxkk5.amplifyapp.com |
src/components/Settings/integrations/Organization/ConnectOrganization.tsx
Outdated
Show resolved
Hide resolved
src/components/Settings/integrations/Organization/ConnectOrganization.tsx
Outdated
Show resolved
Hide resolved
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 have a few suggestions, but I'm going to go ahead and approve.
value={''} | ||
disabled={disabled} | ||
image={ | ||
<img | ||
src="/images/settings-preferences-intergrations-key.png" | ||
alt="The Key" | ||
src="/images/settings-preferences-intergrations-okta.png" |
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.
Can we delete settings-preferences-intergrations-key.png
now?
FYI, there's a typo in the name intergrations
--> integrations
, but all the other images have the typo too.
dca7764
to
7772d0f
Compare
Description
In this PR, I change the "The Key" accordion wording on the Integrations page to "Okta". I also update the ministry organization warning when adding a ministry organization.
With the "The Key" change, I renamed files, folders and components, but both of these changes are pretty simple.
MPDX-7911
MPDX-8514
HelpScout
Checklist: