Skip to content

Commit c7ae7f8

Browse files
committed
refactor: Updates in Textbook
1 parent b031dd5 commit c7ae7f8

3 files changed

Lines changed: 10 additions & 11 deletions

File tree

src/CourseAuthoringRoutes.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ const CourseAuthoringRoutes = () => {
144144
/>
145145
<Route
146146
path="textbooks"
147-
element={<PageWrap><Textbooks courseId={courseId} /></PageWrap>}
147+
element={<PageWrap><Textbooks /></PageWrap>}
148148
/>
149149
</Routes>
150150
</CourseAuthoringPage>

src/textbooks/Textbook.test.jsx

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,18 @@ import { fetchTextbooksQuery } from './data/thunk';
99
import { textbooksMock } from './__mocks__';
1010
import { Textbooks } from '.';
1111
import messages from './messages';
12+
import { CourseAuthoringProvider } from '@src/CourseAuthoringContext';
1213

1314
let axiosMock;
1415
let store;
1516
const courseId = 'course-v1:org+101+101';
1617
const emptyTextbooksMock = { textbooks: [] };
1718

18-
const renderComponent = () => render(<Textbooks courseId={courseId} />);
19+
const renderComponent = () => render(
20+
<CourseAuthoringProvider courseId={courseId}>
21+
<Textbooks />
22+
</CourseAuthoringProvider>
23+
);
1924

2025
describe('<Textbooks />', () => {
2126
beforeEach(async () => {

src/textbooks/Textbooks.jsx

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import PropTypes from 'prop-types';
21
import { useIntl } from '@edx/frontend-platform/i18n';
32
import {
43
Breadcrumb,
@@ -11,11 +10,11 @@ import { Add as AddIcon } from '@openedx/paragon/icons';
1110
import { useSelector } from 'react-redux';
1211
import { Helmet } from 'react-helmet';
1312
import { Link } from 'react-router-dom';
13+
import { useCourseAuthoringContext } from '@src/CourseAuthoringContext';
1414

1515
import { useWaffleFlags } from '../data/apiHooks';
1616
import { SavingErrorAlert } from '../generic/saving-error-alert';
1717
import { getProcessingNotification } from '../generic/processing-notification/data/selectors';
18-
import { useModel } from '../generic/model-store';
1918
import { LoadingSpinner } from '../generic/Loading';
2019
import SubHeader from '../generic/sub-header/SubHeader';
2120
import ConnectionErrorAlert from '../generic/ConnectionErrorAlert';
@@ -28,12 +27,11 @@ import { useTextbooks } from './hooks';
2827
import { getTextbookFormInitialValues } from './utils';
2928
import messages from './messages';
3029

31-
const Textbooks = ({ courseId }) => {
30+
const Textbooks = () => {
3231
const intl = useIntl();
32+
const { courseId, courseDetails } = useCourseAuthoringContext();
3333
const waffleFlags = useWaffleFlags(courseId);
3434

35-
const courseDetails = useModel('courseDetails', courseId);
36-
3735
const {
3836
textbooks,
3937
isLoading,
@@ -156,8 +154,4 @@ const Textbooks = ({ courseId }) => {
156154
);
157155
};
158156

159-
Textbooks.propTypes = {
160-
courseId: PropTypes.string.isRequired,
161-
};
162-
163157
export default Textbooks;

0 commit comments

Comments
 (0)