fix: remove new and legacy from discussion app names (#260)

Both providers should have the same name.
This commit is contained in:
Kshitij Sobti
2022-03-01 12:24:33 +05:30
committed by GitHub
parent f266919cd4
commit d6421da561
3 changed files with 4 additions and 4 deletions

View File

@@ -147,7 +147,7 @@ describe('DiscussionsSettings', () => {
// content has been loaded - prior to proceeding with our expectations.
await waitForElementToBeRemoved(screen.getByRole('status'));
userEvent.click(queryByLabelText(container, 'Select edX (Legacy)'));
userEvent.click(queryByLabelText(container, 'Select edX'));
userEvent.click(queryByText(container, messages.nextButton.defaultMessage));
await waitForElementToBeRemoved(screen.getByRole('status'));

View File

@@ -82,12 +82,12 @@ const messages = defineMessages({
},
'appName-legacy': {
id: 'authoring.discussions.appConfigForm.appName-legacy',
defaultMessage: 'edX (Legacy)',
defaultMessage: 'edX',
description: 'The name of the Legacy edX Discussions app.',
},
'appName-openedx': {
id: 'authoring.discussions.appConfigForm.appName-openedx',
defaultMessage: 'edX (NEW)',
defaultMessage: 'edX',
description: 'The name of the new edX Discussions app.',
},
divisionByGroup: {

View File

@@ -48,7 +48,7 @@ const messages = defineMessages({
// Legacy
'appName-legacy': {
id: 'authoring.discussions.appList.appName-legacy',
defaultMessage: 'edX (Legacy)',
defaultMessage: 'edX',
description: 'The name of the Legacy edX Discussions app.',
},
'appDescription-legacy': {