From 147b4c5b1f09b58a4c8441575543bc853bee92bf Mon Sep 17 00:00:00 2001 From: Patrik Oldsberg Date: Wed, 10 Mar 2021 15:47:32 +0100 Subject: [PATCH] user-settings: avoid depending on descriptions from ApiRefs Co-authored-by: Juan Lulkin Signed-off-by: Patrik Oldsberg --- .changeset/fair-students-buy.md | 5 +++++ .../components/AuthProviders/AuthProviders.test.tsx | 4 +++- .../AuthProviders/DefaultProviderSettings.tsx | 12 ++++++------ 3 files changed, 14 insertions(+), 7 deletions(-) create mode 100644 .changeset/fair-students-buy.md diff --git a/.changeset/fair-students-buy.md b/.changeset/fair-students-buy.md new file mode 100644 index 0000000000..600c86e6c1 --- /dev/null +++ b/.changeset/fair-students-buy.md @@ -0,0 +1,5 @@ +--- +'@backstage/plugin-user-settings': patch +--- + +Avoid using `ApiRef` descriptions in the UI. diff --git a/plugins/user-settings/src/components/AuthProviders/AuthProviders.test.tsx b/plugins/user-settings/src/components/AuthProviders/AuthProviders.test.tsx index 072c90ddac..e299b30594 100644 --- a/plugins/user-settings/src/components/AuthProviders/AuthProviders.test.tsx +++ b/plugins/user-settings/src/components/AuthProviders/AuthProviders.test.tsx @@ -64,7 +64,9 @@ describe('', () => { expect(rendered.getByText('Google')).toBeInTheDocument(); expect( - rendered.getByText(googleAuthApiRef.description), + rendered.getByText( + 'Provides authentication towards Google APIs and identities', + ), ).toBeInTheDocument(); const button = rendered.getByTitle('Sign in to Google'); diff --git a/plugins/user-settings/src/components/AuthProviders/DefaultProviderSettings.tsx b/plugins/user-settings/src/components/AuthProviders/DefaultProviderSettings.tsx index 415d1eaad6..b9c7b763b6 100644 --- a/plugins/user-settings/src/components/AuthProviders/DefaultProviderSettings.tsx +++ b/plugins/user-settings/src/components/AuthProviders/DefaultProviderSettings.tsx @@ -34,7 +34,7 @@ export const DefaultProviderSettings = ({ configuredProviders }: Props) => ( {configuredProviders.includes('google') && ( @@ -42,7 +42,7 @@ export const DefaultProviderSettings = ({ configuredProviders }: Props) => ( {configuredProviders.includes('microsoft') && ( @@ -50,7 +50,7 @@ export const DefaultProviderSettings = ({ configuredProviders }: Props) => ( {configuredProviders.includes('github') && ( @@ -58,7 +58,7 @@ export const DefaultProviderSettings = ({ configuredProviders }: Props) => ( {configuredProviders.includes('gitlab') && ( @@ -66,7 +66,7 @@ export const DefaultProviderSettings = ({ configuredProviders }: Props) => ( {configuredProviders.includes('okta') && ( @@ -74,7 +74,7 @@ export const DefaultProviderSettings = ({ configuredProviders }: Props) => ( {configuredProviders.includes('oauth2') && (