-
Notifications
You must be signed in to change notification settings - Fork 179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
get_appointment_groups is broken #678
Comments
Can you try a couple things?
|
I have followed your suggestions and have managed to make it work. Thanks so much! |
Do you mind explaining what you ended up doing? This may be something that could be cleared up by some more information being added to the docs. |
Ping @eric-vader - Any update on what you did to get it to work? If it was one of those request parameters, it might be worth documenting. |
For |
Thanks. I am going to mark this as a bug because the |
… groups Fixes ucfopen#678 Two issues in one: 1. `canvas.get_appointment_groups()` returns an empty list for teachers because it defaults to "reservable", but instructors can't reserve, so the list is empty. Create a new param on the method that defaults to `reservable` but can be overridden. Update the docstring to note the change. 2. Update `create_appointment_group()` to take either a course ID or a `Course` object. Check that the `context_codes` key is present on the `appointment_group` dict before checking formatting. All tests passing.
Describe the bug
get_appointment_groups
API is broken.To Reproduce
Steps to reproduce the behavior:
canvas.get_appointment_groups()
does not yield results (ie. it is empty)Although,
canvas.get_appointment_group(id=id)
does work as expected.Expected behavior
canvas.get_appointment_groups()
should not return an empty array and should return the appointment group obj.Environment information
python --version
) Python 3.12.4pip show canvasapi
) 3.3.0Additional context
I couldnt get
create_appointment_group
to work properly as well (cannot create appointment), not sure if it is related.The text was updated successfully, but these errors were encountered: