Skip to content

Commit e9f0a65

Browse files
committed
chore: update to paragon 19.1.0
Replace our custom Tour component with the drop-in ProductTour replacement in paragon.
1 parent 7049445 commit e9f0a65

16 files changed

+8
-1242
lines changed

docs/decisions/0010-tour-structure-decisions.md

-32
This file was deleted.

package-lock.json

+3-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636
"@edx/frontend-enterprise-utils": "1.1.1",
3737
"@edx/frontend-lib-special-exams": "1.15.5",
3838
"@edx/frontend-platform": "1.14.3",
39-
"@edx/paragon": "19.0.0",
39+
"@edx/paragon": "19.1.0",
4040
"@edx/frontend-component-header": "^2.4.2",
4141
"@fortawesome/fontawesome-svg-core": "1.2.36",
4242
"@fortawesome/free-brands-svg-icons": "5.15.4",

src/index.scss

-1
Original file line numberDiff line numberDiff line change
@@ -378,7 +378,6 @@
378378
@import "course-home/progress-tab/course-completion/CompletionDonutChart.scss";
379379
@import "course-home/progress-tab/grades/course-grade/GradeBar.scss";
380380
@import "courseware/course/course-exit/CourseRecommendations";
381-
@import "src/tour/Checkpoint.scss";
382381

383382
/** [MM-P2P] Experiment */
384383
@import "experiments/mm-p2p/index.scss";

src/product-tours/ProductTours.jsx

+3-4
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,7 @@ import { useDispatch, useSelector } from 'react-redux';
33
import PropTypes from 'prop-types';
44
import { sendTrackEvent } from '@edx/frontend-platform/analytics';
55
import { getAuthenticatedUser } from '@edx/frontend-platform/auth';
6-
7-
import Tour from '../tour/Tour';
6+
import { ProductTour } from '@edx/paragon';
87

98
import abandonTour from './AbandonTour';
109
import coursewareTour from './CoursewareTour';
@@ -82,7 +81,7 @@ function ProductTours({
8281
}
8382
}, [showNewUserCourseHomeTour]);
8483

85-
// The <Tour /> component cannot handle rendering multiple enabled tours at once.
84+
// The <ProductTour /> component cannot handle rendering multiple enabled tours at once.
8685
// I.e. when adding new tours, beware that if multiple tours are enabled,
8786
// the first enabled tour in the following array will be the only one that renders.
8887
// The suggestion for populating these tour objects is to ensure only one tour is enabled at a time.
@@ -142,7 +141,7 @@ function ProductTours({
142141

143142
return (
144143
<>
145-
<Tour
144+
<ProductTour
146145
tours={tours}
147146
/>
148147
<NewUserCourseHomeTourModal

src/product-tours/ProductTours.test.jsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -306,7 +306,7 @@ describe('Courseware Tour', () => {
306306

307307
expect(global.location.href).toEqual(`http://localhost/course/${courseId}/${defaultSequenceBlock.id}/${unitBlocks[1].id}`);
308308

309-
const checkpoint = container.querySelectorAll('#checkpoint');
309+
const checkpoint = container.querySelectorAll('#pgn__checkpoint');
310310
expect(checkpoint).toHaveLength(showCoursewareTour ? 1 : 0);
311311
});
312312
});

src/tour/Checkpoint.jsx

-139
This file was deleted.

src/tour/Checkpoint.scss

-103
This file was deleted.

src/tour/CheckpointActionRow.jsx

-55
This file was deleted.

0 commit comments

Comments
 (0)