Skip to content
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

Add ability to switch between courses in assignment list extension #563

Merged
merged 2 commits into from
Sep 25, 2016

Conversation

jhamrick
Copy link
Member

This partially (but not fully) address #544. It adds a dropdown menu to the assignment list extension that allows students to switch between different classes:

I still need to add tests for this (and current tests are probably going to fail).

cc @dsblank @randy3k @kthyng @fcurrin

@jhamrick jhamrick added this to the 0.4.0 milestone Sep 25, 2016
@jhamrick
Copy link
Member Author

I should mention that this doesn't do anything regarding access control -- so all students will still be able to see all classes. This should just hopefully make it easier for students to be able to ignore classes that they're not enrolled in. Doing the access control is going to be trickier, so I am going to try to address that in a separate PR. I think this will still be useful even after that though: if a student is in multiple classes they will be able to switch between them and can see individually what they've submitted for each class.

@dsblank
Copy link
Member

dsblank commented Sep 25, 2016

Great! I'm going to use this as soon as it is ready... I have 50% of my students in multiple classes, and they are constantly having to edit their config file. This will lessen the cognitive load.

@jhamrick jhamrick merged commit fa9c3eb into jupyter:master Sep 25, 2016
@jhamrick jhamrick deleted the switch-courses branch September 25, 2016 23:13
@jhamrick
Copy link
Member Author

Ok, should be ready to use! Let me know if there's anything about it that doesn't quite work the way you expect, or if you run into any issues with it.

@fcurrin
Copy link

fcurrin commented Jan 30, 2017

Hi, I was wondering if this works with the newest release of nbgrader. We just updated to 0.3.3 and would like to retain this capability but don't want to replace the files if that would interfere with other changes made in the update.

@jhamrick
Copy link
Member Author

jhamrick commented Feb 2, 2017

This will work with the upcoming release (which you can install with the --pre flag), which is 0.4.0.

don't want to replace the files if that would interfere with other changes made in the update.

Sorry, I'm not sure I understand the issue. Could you elaborate?

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.

3 participants