Skip to content

Commit 6304aea

Browse files
rpenidorpenido
committed
fix: remove deprecated settings waffle flags
1 parent 0c59b78 commit 6304aea

9 files changed

Lines changed: 24 additions & 46 deletions

File tree

‎src/course-checklist/ChecklistSection/ChecklistItemBody.jsx‎

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,24 +5,21 @@ import { ActionRow, Button, Icon } from '@openedx/paragon';
55
import { CheckCircle, RadioButtonUnchecked } from '@openedx/paragon/icons';
66
import { getConfig } from '@edx/frontend-platform';
77

8-
import { useWaffleFlags } from '../../data/apiHooks';
8+
import { useWaffleFlags } from '@src/data/apiHooks';
9+
910
import messages from './messages';
1011

1112
const getUpdateLinks = (courseId, waffleFlags) => {
1213
const baseUrl = getConfig().STUDIO_BASE_URL;
13-
const isLegacyGradingUrl = !waffleFlags.useNewGradingPage;
1414
const isLegacyCertificateUrl = !waffleFlags.useNewCertificatesPage;
15-
const isLegacyCourseDatesUrl = !waffleFlags.useNewScheduleDetailsPage;
1615
const isLegacyOutlineUrl = !waffleFlags.useNewCourseOutlinePage;
1716

1817
return {
1918
welcomeMessage: `/course/${courseId}/course_info`,
20-
gradingPolicy: isLegacyGradingUrl
21-
? `${baseUrl}/settings/grading/${courseId}` : `/course/${courseId}/settings/grading`,
19+
gradingPolicy: `/course/${courseId}/settings/grading`,
2220
certificate: isLegacyCertificateUrl
2321
? `${baseUrl}/certificates/${courseId}` : `/course/${courseId}/certificates`,
24-
courseDates: isLegacyCourseDatesUrl
25-
? `${baseUrl}/settings/details/${courseId}#schedule` : `/course/${courseId}/settings/details/#schedule`,
22+
courseDates: `/course/${courseId}/settings/details/#schedule`,
2623
proctoringEmail: `${baseUrl}/pages-and-resources/proctoring/settings`,
2724
outline: isLegacyOutlineUrl ? `${baseUrl}/course/${courseId}` : `/course/${courseId}`,
2825
};

‎src/course-checklist/ChecklistSection/ChecklistSection.test.jsx‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,9 @@ import { camelCaseObject } from '@edx/frontend-platform';
22

33
import {
44
initializeMocks, render, screen, within,
5-
} from '../../testUtils';
6-
import { getApiWaffleFlagsUrl } from '../../data/api';
5+
} from '@src/testUtils';
6+
import { getApiWaffleFlagsUrl } from '@src/data/api';
7+
78
import { generateCourseLaunchData } from '../factories/mockApiResponses';
89
import { checklistItems } from './utils/courseChecklistData';
910
import messages from './messages';
@@ -36,9 +37,7 @@ describe('ChecklistSection', () => {
3637
axiosMock
3738
.onGet(getApiWaffleFlagsUrl(courseId))
3839
.reply(200, {
39-
useNewGradingPage: true,
4040
useNewCertificatesPage: true,
41-
useNewScheduleDetailsPage: true,
4241
useNewCourseOutlinePage: true,
4342
});
4443
});

‎src/course-outline/status-bar/LegacyStatusBar.tsx‎

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,16 +5,16 @@ import {
55
Button, Hyperlink, Form, Stack, useToggle,
66
} from '@openedx/paragon';
77
import { Link } from 'react-router-dom';
8+
import { type ReactNode } from 'react';
89

9-
import { ReactNode } from 'react';
1010
import { CourseOutlineStatusBar } from '@src/course-outline/data/types';
1111
import { ContentTagsDrawerSheet } from '@src/content-tags-drawer';
1212
import TagCount from '@src/generic/tag-count';
1313
import { useHelpUrls } from '@src/help-urls/hooks';
14-
import { useWaffleFlags } from '@src/data/apiHooks';
1514
import { VIDEO_SHARING_OPTIONS } from '@src/course-outline/constants';
1615
import { useContentTagsCount } from '@src/generic/data/apiHooks';
1716
import { getVideoSharingOptionText } from '@src/course-outline/utils';
17+
1818
import messages from './messages';
1919

2020
interface StatusBarItemProps {
@@ -47,7 +47,6 @@ export const LegacyStatusBar = ({
4747
handleVideoSharingOptionChange,
4848
}: LegacyStatusBarProps) => {
4949
const intl = useIntl();
50-
const waffleFlags = useWaffleFlags(courseId);
5150

5251
const {
5352
courseReleaseDate,
@@ -67,7 +66,6 @@ export const LegacyStatusBar = ({
6766

6867
const courseReleaseDateObj = moment.utc(courseReleaseDate, 'MMM DD, YYYY [at] HH:mm UTC', true);
6968
const checkListTitle = `${completedCourseLaunchChecks + completedCourseBestPracticesChecks}/${totalCourseLaunchChecks + totalCourseBestPracticesChecks}`;
70-
const scheduleDestination = () => new URL(`settings/details/${courseId}#schedule`, getConfig().STUDIO_BASE_URL).href;
7169

7270
const {
7371
contentHighlights: contentHighlightsUrl,
@@ -88,7 +86,7 @@ export const LegacyStatusBar = ({
8886
<StatusBarItem title={intl.formatMessage(messages.startDateTitle)}>
8987
<Link
9088
className="small"
91-
to={waffleFlags.useNewScheduleDetailsPage ? `/course/${courseId}/settings/details/#schedule` : scheduleDestination()}
89+
to={`/course/${courseId}/settings/details/#schedule`}
9290
>
9391
{courseReleaseDateObj.isValid() ? (
9492
<FormattedDate

‎src/course-outline/status-bar/StatusBar.tsx‎

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,15 @@
11
import moment, { Moment } from 'moment/moment';
22
import { FormattedDate, FormattedMessage } from '@edx/frontend-platform/i18n';
3-
import { getConfig } from '@edx/frontend-platform/config';
43
import { Badge, Icon, Stack } from '@openedx/paragon';
54
import { Link } from 'react-router-dom';
6-
7-
import type { ChecklistType, CourseOutlineStatusBar } from '@src/course-outline/data/types';
85
import {
96
Cached, ChecklistRtl, Description, Event,
107
} from '@openedx/paragon/icons';
11-
import { useWaffleFlags } from '@src/data/apiHooks';
8+
9+
import type { ChecklistType, CourseOutlineStatusBar } from '@src/course-outline/data/types';
1210
import { useEntityLinksSummaryByDownstreamContext } from '@src/course-libraries/data/apiHooks';
1311
import { useCourseDetails } from '@src/course-outline/data/apiHooks';
12+
1413
import messages from './messages';
1514
import { NotificationStatusIcon } from './NotificationStatusIcon';
1615

@@ -179,8 +178,6 @@ export const StatusBar = ({
179178
isLoading,
180179
courseId,
181180
}: StatusBarProps) => {
182-
const waffleFlags = useWaffleFlags(courseId);
183-
184181
const {
185182
endDate,
186183
courseReleaseDate,
@@ -189,7 +186,6 @@ export const StatusBar = ({
189186

190187
const courseReleaseDateObj = moment.utc(courseReleaseDate, 'MMM DD, YYYY [at] HH:mm UTC', true);
191188
const endDateObj = moment.utc(endDate);
192-
const scheduleDestination = () => new URL(`settings/details/${courseId}#schedule`, getConfig().STUDIO_BASE_URL).href;
193189

194190
if (isLoading) {
195191
return null;
@@ -203,7 +199,7 @@ export const StatusBar = ({
203199
startDate={courseReleaseDateObj}
204200
endDate={endDateObj}
205201
startDateRaw={courseReleaseDate}
206-
datesLink={waffleFlags.useNewScheduleDetailsPage ? `/course/${courseId}/settings/details/#schedule` : scheduleDestination()}
202+
datesLink={`/course/${courseId}/settings/details/#schedule`}
207203
/>
208204
<Checklists courseId={courseId} checklist={checklist} />
209205
<LibraryUpdates courseId={courseId} />

‎src/custom-pages/CustomPages.test.tsx‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,7 @@ describe('CustomPages', () => {
6262
axiosMock
6363
.onGet(getApiWaffleFlagsUrl(courseId))
6464
.reply(200, {
65-
useNewGradingPage: true,
6665
useNewCertificatesPage: true,
67-
useNewScheduleDetailsPage: true,
6866
useNewCourseOutlinePage: true,
6967
});
7068
});

‎src/data/api.ts‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,17 +76,14 @@ export const waffleFlagDefaults = {
7676
enableCourseOptimizerCheckPrevRunLinks: false,
7777
useNewHomePage: true,
7878
useNewCustomPages: true,
79-
useNewScheduleDetailsPage: true,
8079
useNewAdvancedSettingsPage: true,
81-
useNewGradingPage: true,
8280
useNewUpdatesPage: true,
8381
useNewImportPage: false,
8482
useNewExportPage: true,
8583
useNewFilesUploadsPage: true,
8684
useNewVideoUploadsPage: true,
8785
useNewCourseOutlinePage: true,
8886
useNewUnitPage: false,
89-
useNewCourseTeamPage: true,
9087
useNewCertificatesPage: true,
9188
useNewTextbooksPage: true,
9289
useNewGroupConfigurationsPage: true,

‎src/generic/help-sidebar/HelpSidebar.jsx‎

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,6 @@ const HelpSidebar = ({
3535
return referObj.href;
3636
};
3737

38-
const scheduleAndDetailsDestination = generateLegacyURL(scheduleAndDetails);
39-
const gradingDestination = generateLegacyURL(grading);
40-
const courseTeamDestination = generateLegacyURL(courseTeam);
4138
const advancedSettingsDestination = generateLegacyURL(advancedSettings);
4239
const groupConfigurationsDestination = generateLegacyURL(groupConfigurations);
4340

@@ -78,26 +75,23 @@ const HelpSidebar = ({
7875
<ul className="p-0 mb-0">
7976
{showOtherLink(scheduleAndDetails) && (
8077
<HelpSidebarLink
81-
pathToPage={waffleFlags.useNewScheduleDetailsPage
82-
? `/course/${courseId}/${scheduleAndDetails}` : scheduleAndDetailsDestination}
78+
pathToPage={`/course/${courseId}/${scheduleAndDetails}`}
8379
title={intl.formatMessage(
8480
messages.sidebarLinkToScheduleAndDetails,
8581
)}
86-
isNewPage={waffleFlags.useNewScheduleDetailsPage}
82+
isNewPage
8783
/>
8884
)}
8985
{showOtherLink(grading) && (
9086
<HelpSidebarLink
91-
pathToPage={waffleFlags.useNewGradingPage
92-
? `/course/${courseId}/${grading}` : gradingDestination}
87+
pathToPage={`/course/${courseId}/${grading}`}
9388
title={intl.formatMessage(messages.sidebarLinkToGrading)}
9489
isNewPage={waffleFlags.useNewGradingPage}
9590
/>
9691
)}
9792
{showOtherLink(courseTeam) && (
9893
<HelpSidebarLink
99-
pathToPage={waffleFlags.useNewCourseTeamPage
100-
? `/course/${courseId}/${courseTeam}` : courseTeamDestination}
94+
pathToPage={`/course/${courseId}/${courseTeam}`}
10195
title={intl.formatMessage(messages.sidebarLinkToCourseTeam)}
10296
isNewPage={waffleFlags.useNewCourseTeamPage}
10397
/>

‎src/header/hooks.tsx‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,15 +84,15 @@ export const useSettingMenuItems = (courseId: string) => {
8484

8585
const items = [
8686
{
87-
href: waffleFlags.useNewScheduleDetailsPage ? `/course/${courseId}/settings/details` : `${studioBaseUrl}/settings/details/${courseId}`,
87+
href: `/course/${courseId}/settings/details`,
8888
title: intl.formatMessage(messages['header.links.scheduleAndDetails']),
8989
},
9090
{
91-
href: waffleFlags.useNewGradingPage ? `/course/${courseId}/settings/grading` : `${studioBaseUrl}/settings/grading/${courseId}`,
91+
href: `/course/${courseId}/settings/grading`,
9292
title: intl.formatMessage(messages['header.links.grading']),
9393
},
9494
{
95-
href: waffleFlags.useNewCourseTeamPage ? `/course/${courseId}/course_team` : `${studioBaseUrl}/course_team/${courseId}`,
95+
href: `/course/${courseId}/course_team`,
9696
title: intl.formatMessage(messages['header.links.courseTeam']),
9797
},
9898
{

‎src/pages-and-resources/pages/PageCard.test.jsx‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,14 @@
11
import { getConfig } from '@edx/frontend-platform';
22

3+
import { getApiWaffleFlagsUrl } from '@src/data/api';
34
import {
45
initializeMocks,
56
screen,
67
render,
78
waitFor,
8-
} from '../../testUtils';
9+
} from '@src/testUtils';
10+
911
import PageGrid from './PageGrid';
10-
import { getApiWaffleFlagsUrl } from '../../data/api';
1112

1213
import PagesAndResourcesProvider from '../PagesAndResourcesProvider';
1314

@@ -49,9 +50,7 @@ describe('LiveSettings', () => {
4950
axiosMock
5051
.onGet(getApiWaffleFlagsUrl(courseId))
5152
.reply(200, {
52-
useNewGradingPage: true,
5353
useNewCertificatesPage: true,
54-
useNewScheduleDetailsPage: true,
5554
useNewCourseOutlinePage: true,
5655
});
5756
});

0 commit comments

Comments
 (0)