Andal.LND/frontend-app-authoring
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Rename builtin discussion providers, "edX" -> "Open edX" (#2660)

Browse Source
This commit is contained in:
Kyle McCormick
2025-11-18 09:55:33 -05:00
committed by GitHub
parent daed664404
commit 5fadccabe2
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/pages-and-resources/discussions/DiscussionsSettings.test.jsx
View File

@@ -147,7 +147,7 @@ describe('DiscussionsSettings', () => {
// content has been loaded - prior to proceeding with our expectations.
await waitForElementToBeRemoved(screen.queryByRole('status'));
await user.click(queryByLabelText(container, 'Select edX'));
await user.click(queryByLabelText(container, 'Select Open edX (legacy)'));
await user.click(queryByText(container, messages.nextButton.defaultMessage));
expect(queryByTestId(container, 'appList')).not.toBeInTheDocument();

8
src/pages-and-resources/discussions/app-config-form/messages.js
View File

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