Skip to content

Commit 7b98435

Browse files
committed
revert: update plugin-slots version
1 parent 2e8e6dc commit 7b98435

9 files changed

Lines changed: 15 additions & 28 deletions

File tree

src/courseware/course/sequence/Unit/index.test.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ describe('<Unit />', () => {
5656
describe('unit title', () => {
5757
it('has two children', () => {
5858
renderComponent(defaultProps);
59-
const unitTitleWrapper = screen.getByTestId('org.openedx.frontend.learning.unit_title.v2').children[0];
59+
const unitTitleWrapper = screen.getByTestId('org.openedx.frontend.learning.unit_title.v1').children[0];
6060

6161
expect(unitTitleWrapper.children).toHaveLength(3);
6262
});

src/plugin-slots/CourseBreadcrumbsSlot/README.md

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Course Breadcrumbs Slot
22

3-
### Slot ID: `org.openedx.frontend.learning.course_breadcrumbs.v2`
3+
### Slot ID: `org.openedx.frontend.learning.course_breadcrumbs.v1`
44

55
### Slot ID Aliases
66
* `course_breadcrumbs_slot`
@@ -25,7 +25,7 @@ import CourseBreadcrumbs from './src/courseware/course/breadcrumbs';
2525

2626
const config = {
2727
pluginSlots: {
28-
'org.openedx.frontend.learning.course_breadcrumbs.v2': {
28+
'org.openedx.frontend.learning.course_breadcrumbs.v1': {
2929
keepDefault: false,
3030
plugins: [
3131
{
@@ -62,7 +62,7 @@ import { DIRECT_PLUGIN, PLUGIN_OPERATIONS } from '@openedx/frontend-plugin-frame
6262

6363
const config = {
6464
pluginSlots: {
65-
'org.openedx.frontend.learning.course_breadcrumbs.v2': {
65+
'org.openedx.frontend.learning.course_breadcrumbs.v1': {
6666
keepDefault: false,
6767
plugins: [
6868
{
@@ -82,7 +82,3 @@ const config = {
8282

8383
export default config;
8484
```
85-
86-
## Version Notes
87-
88-
- `v2`: Removed default slot content.

src/plugin-slots/CourseBreadcrumbsSlot/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ export const CourseBreadcrumbsSlot : React.FC<Props> = ({
1414
courseId, sectionId, sequenceId, unitId, isStaff,
1515
}) => (
1616
<PluginSlot
17-
id="org.openedx.frontend.learning.course_breadcrumbs.v2"
17+
id="org.openedx.frontend.learning.course_breadcrumbs.v1"
1818
idAliases={['course_breadcrumbs_slot']}
1919
slotOptions={{
2020
mergeProps: true,

src/plugin-slots/README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
* [`org.openedx.frontend.layout.footer.v1`](./FooterSlot/)
44
* [`org.openedx.frontend.layout.header_learning.v1`](./HeaderSlot/)
55
* [`org.openedx.frontend.learning.content_iframe_loader.v1`](./ContentIFrameLoaderSlot/)
6-
* [`org.openedx.frontend.learning.course_breadcrumbs.v2`](./CourseBreadcrumbsSlot/)
6+
* [`org.openedx.frontend.learning.course_breadcrumbs.v1`](./CourseBreadcrumbsSlot/)
77
* [`org.openedx.frontend.learning.course_home_section_outline.v1`](./CourseHomeSectionOutlineSlot/)
88
* [`org.openedx.frontend.learning.course_outline_mobile_sidebar_trigger.v1`](./CourseOutlineMobileSidebarTriggerSlot/)
99
* [`org.openedx.frontend.learning.course_outline_sidebar_trigger.v1`](./CourseOutlineSidebarTriggerSlot)
@@ -23,5 +23,5 @@
2323
* [`org.openedx.frontend.learning.progress_tab_grade_breakdown.v1`](./ProgressTabGradeBreakdownSlot/)
2424
* [`org.openedx.frontend.learning.progress_tab_related_links.v1`](./ProgressTabRelatedLinksSlot/)
2525
* [`org.openedx.frontend.learning.sequence_container.v1`](./SequenceContainerSlot/)
26-
* [`org.openedx.frontend.learning.sequence_navigation.v2`](./SequenceNavigationSlot/)
27-
* [`org.openedx.frontend.learning.unit_title.v2`](./UnitTitleSlot/)
26+
* [`org.openedx.frontend.learning.sequence_navigation.v1`](./SequenceNavigationSlot/)
27+
* [`org.openedx.frontend.learning.unit_title.v1`](./UnitTitleSlot/)

src/plugin-slots/SequenceNavigationSlot/README.md

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Sequence Navigation Slot
22

3-
### Slot ID: `org.openedx.frontend.learning.sequence_navigation.v2`
3+
### Slot ID: `org.openedx.frontend.learning.sequence_navigation.v1`
44

55
### Props:
66
* `sequenceId` (string) — Current sequence identifier
@@ -29,7 +29,7 @@ import { SequenceNavigation } from './src/courseware/course/sequence/sequence-na
2929

3030
const config = {
3131
pluginSlots: {
32-
'org.openedx.frontend.learning.sequence_navigation.v2': {
32+
'org.openedx.frontend.learning.sequence_navigation.v1': {
3333
keepDefault: false,
3434
plugins: [
3535
{
@@ -66,7 +66,7 @@ import { DIRECT_PLUGIN, PLUGIN_OPERATIONS } from '@openedx/frontend-plugin-frame
6666

6767
const config = {
6868
pluginSlots: {
69-
'org.openedx.frontend.learning.sequence_navigation.v2': {
69+
'org.openedx.frontend.learning.sequence_navigation.v1': {
7070
keepDefault: false,
7171
plugins: [
7272
{
@@ -116,7 +116,3 @@ const config = {
116116

117117
export default config;
118118
```
119-
120-
## Version Notes
121-
122-
- `v2`: Removed default slot content.

src/plugin-slots/SequenceNavigationSlot/index.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const SequenceNavigationSlot = ({
1010
previousHandler,
1111
}) => (
1212
<PluginSlot
13-
id="org.openedx.frontend.learning.sequence_navigation.v2"
13+
id="org.openedx.frontend.learning.sequence_navigation.v1"
1414
slotOptions={{
1515
mergeProps: true,
1616
}}

src/plugin-slots/UnitTitleSlot/README.md

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Unit Title Slot
22

3-
### Slot ID: `org.openedx.frontend.learning.unit_title.v2`
3+
### Slot ID: `org.openedx.frontend.learning.unit_title.v1`
44

55
### Slot ID Aliases
66
* `unit_title_slot`
@@ -27,7 +27,7 @@ import { DIRECT_PLUGIN, PLUGIN_OPERATIONS } from '@openedx/frontend-plugin-frame
2727

2828
const config = {
2929
pluginSlots: {
30-
'org.openedx.frontend.learning.unit_title.v2': {
30+
'org.openedx.frontend.learning.unit_title.v1': {
3131
plugins: [
3232
{
3333
// Insert custom content after unit title
@@ -51,8 +51,3 @@ const config = {
5151

5252
export default config;
5353
```
54-
55-
## Version Notes
56-
57-
- `v2`: Removed default slot content. `isEnabledOutlineSidebar` is no longer used internally,
58-
but still passed as a plugin prop (`true`) to maintain compatibility.

src/plugin-slots/UnitTitleSlot/index.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ const UnitTitleSlot = ({
1515

1616
return (
1717
<PluginSlot
18-
id="org.openedx.frontend.learning.unit_title.v2"
18+
id="org.openedx.frontend.learning.unit_title.v1"
1919
idAliases={['unit_title_slot']}
2020
pluginProps={{
2121
unitId,

webpack.dev-tutor.config.js

Whitespace-only changes.

0 commit comments

Comments
 (0)