Skip to content

Remove user_lists, learning_path_parents entirely from serializers#2922

Open
mbertrand wants to merge 6 commits intomainfrom
mb/remove_blank_list_fields
Open

Remove user_lists, learning_path_parents entirely from serializers#2922
mbertrand wants to merge 6 commits intomainfrom
mb/remove_blank_list_fields

Conversation

@mbertrand
Copy link
Member

What are the relevant tickets?

Closes https://github.com/mitodl/hq/issues/10089

Description (What does it do?)

Removes the learning_path_parents and user_list_parents fields entirely from learning resource serializers.
This will cause lots of failures on the openapi test but they can be ignored.

How can this be tested?

  • Log in as an admin, go to the search page, add some courses to your Favorites and learning paths, refresh the page. You should see courses in your paths/lists marked as such before and after the refresh.
  • Go to http://open.odl.local:8062/learningpaths. You should still be able to see your learning paths, the list of resources in each, etc.
  • Go to http://open.odl.local:8062/dashboard/my-lists. You should still be able to see your userlists, the list of resources in each, etc.

@mbertrand mbertrand added Needs Review An open Pull Request that is ready for review and removed Needs Review An open Pull Request that is ready for review labels Feb 4, 2026
@mbertrand mbertrand force-pushed the mb/remove_blank_list_fields branch from 3e52478 to ac2624e Compare February 5, 2026 17:06
@mbertrand mbertrand force-pushed the mb/remove_blank_list_fields branch from ac2624e to 1d0251d Compare February 5, 2026 19:23
@mbertrand mbertrand force-pushed the mb/remove_blank_list_fields branch 2 times, most recently from d7c2d46 to dba528e Compare February 6, 2026 13:13
@mbertrand mbertrand force-pushed the mb/remove_blank_list_fields branch from dba528e to c46c5a0 Compare February 6, 2026 13:16
@mbertrand mbertrand force-pushed the mb/remove_blank_list_fields branch from c46c5a0 to 3bea23a Compare February 6, 2026 13:45
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.

1 participant