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

Must work with IDDE #5

Open
pdpinch opened this issue Apr 30, 2014 · 4 comments
Open

Must work with IDDE #5

pdpinch opened this issue Apr 30, 2014 · 4 comments
Assignees

Comments

@pdpinch
Copy link

pdpinch commented Apr 30, 2014

  • Instructor must be able to grant an extension through the IDDE interface
  • When an extension is granted, student should be able to upload a file
  • When an extension is removed, and the due date has passed, student should no longer be able to upload
@chrisrossi
Copy link

Currently the SGA itself seems to work fine with IDDE, but IDDE itself isn't working correctly. When an extended due date is set on a homework, it doesn't cascade down the tree to individual problems, the way it should. It may be that IDDE has experienced some bitrot with no one specifically focused on it, or it may not be interacting the XBlocks correctly.

@chrisrossi
Copy link

Indeed, what I was observing was a bug in IDDE. I have submitted a pull request to fix this issue: https://github.com/edx/edx-platform/pull/3718

Note that an earlier pull request I submitted related to this project has so far received no feedback:
https://github.com/edx/edx-platform/pull/3615

Given that, MIT may need to make themselves a little bit of a squeaky wheel to make sure this fix goes out in a timely manner.

@pdpinch pdpinch assigned carsongee and unassigned chrisrossi May 19, 2014
@pdpinch
Copy link
Author

pdpinch commented May 19, 2014

This still isn't working for me, but I'm assuming that's because Carson's doesn't have the IDDE fix in edx/edx-platform#3718

I'd like to leave this open until I can confirm it's fixed.

@pdpinch pdpinch reopened this May 19, 2014
@carsongee
Copy link

I have code running now with the patch applied

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants