Skip to content

feat: New link on studio home to Taxonomy list - #23

Merged
andrey-canon merged 2 commits into
open-release/palm.nelpfrom
and/backport_taxonomy_tab
May 15, 2024
Merged

feat: New link on studio home to Taxonomy list#23
andrey-canon merged 2 commits into
open-release/palm.nelpfrom
and/backport_taxonomy_tab

Conversation

@andrey-canon

@andrey-canon andrey-canon commented May 7, 2024

Copy link
Copy Markdown
Collaborator

Description

This is a backport of openedx#33439

Testing instructions

The provided instructions on the original pr are enough, just consider that you can use the branch master of course-authoring and if you are using a cms custom theme, you should update that

@andrey-canon
andrey-canon force-pushed the and/backport_home_api branch from 342b469 to 23fe279 Compare May 8, 2024 21:51
@johanseto
johanseto self-requested a review May 8, 2024 21:53
@andrey-canon
andrey-canon force-pushed the and/backport_taxonomy_tab branch from 38f3af6 to 447c4b6 Compare May 8, 2024 22:03

@johanseto johanseto left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The template is loading, but seems redirecting to a not good url.

Comment thread cms/djangoapps/contentstore/utils.py Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that course-authoring in master now doesn't have that route.
Peek 2024-05-08 17-47

@andrey-canon andrey-canon May 9, 2024

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are right they removed that two weeks ago openedx/frontend-app-authoring#954

@andrey-canon
andrey-canon force-pushed the and/backport_home_api branch from 23fe279 to 7d8f3a5 Compare May 9, 2024 23:24
@andrey-canon
andrey-canon force-pushed the and/backport_taxonomy_tab branch 2 times, most recently from 3898b92 to ac06e18 Compare May 9, 2024 23:35
@andrey-canon
andrey-canon requested a review from johanseto May 10, 2024 14:08

@johanseto johanseto left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working =)
Peek 2024-05-10 16-36

@andrey-canon
andrey-canon changed the base branch from and/backport_home_api to open-release/palm.nelp May 14, 2024 19:20
@andrey-canon
andrey-canon force-pushed the and/backport_taxonomy_tab branch from ac06e18 to bcce1d7 Compare May 14, 2024 19:24
@andrey-canon
andrey-canon merged commit 7696b70 into open-release/palm.nelp May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants