Skip to content

Conversation

@odlbot
Copy link
Contributor

@odlbot odlbot commented Aug 29, 2024

Chris Chudzicki

Carey P Gumaer

Nathan Levesque

rhysyngsun and others added 5 commits August 29, 2024 09:47
* WIP

* rework api endpoints and implement them

* remove the modal after we're done with it

* await setting new list parents before dismissing modal

* remove circularprogress for now

* fix tests

* try and fix warning about api parameter

* try and fix python tests again

* remove id parameter because it broke everything

* remove extraneous API endpoints

* simplify api signature

* remove extraneous debugging line

* filter user list manipulation based on the current user and properly rebuild position index

* restrict learning path editing to learning path editor

* add reordering logic to learning path setting too

* fix js tests

* fix schema annotations

* fix annotations and response types

* override get_serializer_class to satisfy checks

* regenerate spec

* fix invalidations

* revise invalidations based on feedback

* only specify parameters where necessary

* fix backwards classes

* add learning path permission class instead of manual check

* add some tests for the new API endpoints

* use get_serializer_class

* properly use get_serializer_class

* throw an unauthorized error if you try to assign courses to userlists that don't belong to you

* fix test by checking for raising of permission error

* fix rebase issue yet again

* add a test to ensure accidental unassignment does not happen

* pass in course to userlist assignment function
* remove pathways link

* require href on nav links
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants