Skip to content

Commit 5fadcca

Browse files
authored
fix: Rename builtin discussion providers, "edX" -> "Open edX" (#2660)
1 parent daed664 commit 5fadcca

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/pages-and-resources/discussions/DiscussionsSettings.test.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ describe('DiscussionsSettings', () => {
147147
// content has been loaded - prior to proceeding with our expectations.
148148
await waitForElementToBeRemoved(screen.queryByRole('status'));
149149

150-
await user.click(queryByLabelText(container, 'Select edX'));
150+
await user.click(queryByLabelText(container, 'Select Open edX (legacy)'));
151151
await user.click(queryByText(container, messages.nextButton.defaultMessage));
152152

153153
expect(queryByTestId(container, 'appList')).not.toBeInTheDocument();

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -107,13 +107,13 @@ const messages = defineMessages({
107107
},
108108
'appName-legacy': {
109109
id: 'authoring.discussions.appConfigForm.appName-legacy',
110-
defaultMessage: 'edX',
111-
description: 'The name of the Legacy edX Discussions app.',
110+
defaultMessage: 'Open edX (legacy)',
111+
description: 'The name of the Legacy Open edX Discussions app.',
112112
},
113113
'appName-openedx': {
114114
id: 'authoring.discussions.appConfigForm.appName-openedx',
115-
defaultMessage: 'edX (new)',
116-
description: 'The name of the new edX Discussions app.',
115+
defaultMessage: 'Open edX',
116+
description: 'The name of the new Open edX Discussions app.',
117117
},
118118
divisionByGroup: {
119119
id: 'authoring.discussions.builtIn.divisionByGroup',

0 commit comments

Comments
 (0)