Use instructor_tasks for canvas work #183
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 2 commits
September 4, 2020 11:02
gsidebo
suggested changes
Sep 8, 2020
There was a problem hiding this comment.
One thing I noticed: the output shows status "Incomplete" for tasks that appear to have completed, and I also see "0 out of 0" in the last column.
I also think it might be helpful to show how long ago the task executed in minutes if it was within the last 6 hours or so (e.g.: "30 minutes ago" instead of the ISO timestamp)
|
How does this look for a message? |
|
|
shaidar
pushed a commit
that referenced
this pull request
Sep 15, 2020
* Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time
asadiqbal08
pushed a commit
that referenced
this pull request
Nov 19, 2020
* Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time
asadiqbal08
pushed a commit
that referenced
this pull request
Nov 19, 2020
* Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time
blarghmatey
pushed a commit
that referenced
this pull request
Nov 19, 2020
* Sync canvas enrollments * Added instructor dashboard button to push edX grades to Canvas * Fixed canvas grade syncing JS * Changed 'edX' reference to 'MITx' * Use EDIT_COURSE_ACCESS permission instead of is_staff * Use OVERRIDE_GRADES rule instead * Use instructor_tasks for canvas work (#183) * Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time * Don't return output to prevent task output max size error (#189) Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]>
blarghmatey
pushed a commit
that referenced
this pull request
Dec 9, 2020
* Sync canvas enrollments * Added instructor dashboard button to push edX grades to Canvas * Fixed canvas grade syncing JS * Changed 'edX' reference to 'MITx' * Use EDIT_COURSE_ACCESS permission instead of is_staff * Use OVERRIDE_GRADES rule instead * Use instructor_tasks for canvas work (#183) * Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time * Don't return output to prevent task output max size error (#189) Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]>
blarghmatey
pushed a commit
that referenced
this pull request
Jan 25, 2021
* Sync canvas enrollments * Added instructor dashboard button to push edX grades to Canvas * Fixed canvas grade syncing JS * Changed 'edX' reference to 'MITx' * Use EDIT_COURSE_ACCESS permission instead of is_staff * Use OVERRIDE_GRADES rule instead * Use instructor_tasks for canvas work (#183) * Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time * Don't return output to prevent task output max size error (#189) Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]>
blarghmatey
pushed a commit
that referenced
this pull request
Feb 8, 2021
* Sync canvas enrollments * Added instructor dashboard button to push edX grades to Canvas * Fixed canvas grade syncing JS * Changed 'edX' reference to 'MITx' * Use EDIT_COURSE_ACCESS permission instead of is_staff * Use OVERRIDE_GRADES rule instead * Use instructor_tasks for canvas work (#183) * Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time * Don't return output to prevent task output max size error (#189) Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]>
blarghmatey
pushed a commit
that referenced
this pull request
Feb 18, 2021
* Sync canvas enrollments * Added instructor dashboard button to push edX grades to Canvas * Fixed canvas grade syncing JS * Changed 'edX' reference to 'MITx' * Use EDIT_COURSE_ACCESS permission instead of is_staff * Use OVERRIDE_GRADES rule instead * Use instructor_tasks for canvas work (#183) * Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time * Don't return output to prevent task output max size error (#189) Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]>
umarmughal824
pushed a commit
that referenced
this pull request
Apr 15, 2021
* Sync canvas enrollments * Added instructor dashboard button to push edX grades to Canvas * Fixed canvas grade syncing JS * Changed 'edX' reference to 'MITx' * Use EDIT_COURSE_ACCESS permission instead of is_staff * Use OVERRIDE_GRADES rule instead * Use instructor_tasks for canvas work (#183) * Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time * Don't return output to prevent task output max size error (#189) Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]>
umarmughal824
pushed a commit
that referenced
this pull request
Apr 20, 2021
* Canvas Commits (#223) * Sync canvas enrollments * Added instructor dashboard button to push edX grades to Canvas * Fixed canvas grade syncing JS * Changed 'edX' reference to 'MITx' * Use EDIT_COURSE_ACCESS permission instead of is_staff * Use OVERRIDE_GRADES rule instead * Use instructor_tasks for canvas work (#183) * Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time * Don't return output to prevent task output max size error (#189) Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]> * Reimplemented legacy dashboard enrollment data functionality (60a6ddc) * Fix syntax error in import statement * Cherry-Picked: Enrolment commits (#228) * Adding enrollment restriction for users based on social auth providers In order to prevent users outside of MIT creating a collaborator account and self-enrolling in courses we need to be able to restrict self-enrollment to users who authenticate using the MIT Kerberos SAML backend. This adds filtering of self-enrollment based on a feature flag that sets the allowed provider ids. (cherry picked from commit a4c53a1) * Bugfix for collaborator account access check Co-authored-by: Tobias Macey <[email protected]> * Fixing import path for canvas_integration plugin * Fix import namespace for relative imprts edX requires the full namespace for imports so importing e.g. `remote_gradebook.tasks` is no longer valid. Instead it requires `from lms.djangoapps.remote_gradebook import tasks`. * #229 change submission type and published state on canvas (#232) - Change submission type to "None" - Change published state to "False" * Fixing encoding of header row for posting to remote gradebook Posting to the remote gradebook via lmod proxy was failing due to a key error when parsing the CSV content. This was due to erroneously encoding a string object. This commit adds a check for the type of the header value to prevent double encoding it. * fix the imports * address feedback Co-authored-by: Asad Iqbal <[email protected]> Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]> Co-authored-by: sar <[email protected]> Co-authored-by: Arslan Ashraf <[email protected]> Co-authored-by: Tobias Macey <[email protected]>
blarghmatey
pushed a commit
that referenced
this pull request
Jun 3, 2021
* Sync canvas enrollments * Added instructor dashboard button to push edX grades to Canvas * Fixed canvas grade syncing JS * Changed 'edX' reference to 'MITx' * Use EDIT_COURSE_ACCESS permission instead of is_staff * Use OVERRIDE_GRADES rule instead * Use instructor_tasks for canvas work (#183) * Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time * Don't return output to prevent task output max size error (#189) Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]>
blarghmatey
added a commit
that referenced
this pull request
Sep 7, 2021
* Canvas Commits (#223) * Sync canvas enrollments * Added instructor dashboard button to push edX grades to Canvas * Fixed canvas grade syncing JS * Changed 'edX' reference to 'MITx' * Use EDIT_COURSE_ACCESS permission instead of is_staff * Use OVERRIDE_GRADES rule instead * Use instructor_tasks for canvas work (#183) * Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time * Don't return output to prevent task output max size error (#189) Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]> * Reimplemented legacy dashboard enrollment data functionality (60a6ddc) * Fix syntax error in import statement * Cherry-Picked: Enrolment commits (#228) * Adding enrollment restriction for users based on social auth providers In order to prevent users outside of MIT creating a collaborator account and self-enrolling in courses we need to be able to restrict self-enrollment to users who authenticate using the MIT Kerberos SAML backend. This adds filtering of self-enrollment based on a feature flag that sets the allowed provider ids. (cherry picked from commit a4c53a1) * Bugfix for collaborator account access check Co-authored-by: Tobias Macey <[email protected]> * Fixing import path for canvas_integration plugin * Fix import namespace for relative imprts edX requires the full namespace for imports so importing e.g. `remote_gradebook.tasks` is no longer valid. Instead it requires `from lms.djangoapps.remote_gradebook import tasks`. * #229 change submission type and published state on canvas (#232) - Change submission type to "None" - Change published state to "False" * Fixing encoding of header row for posting to remote gradebook Posting to the remote gradebook via lmod proxy was failing due to a key error when parsing the CSV content. This was due to erroneously encoding a string object. This commit adds a check for the type of the header value to prevent double encoding it. * fix the imports * address feedback Co-authored-by: Asad Iqbal <[email protected]> Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]> Co-authored-by: sar <[email protected]> Co-authored-by: Arslan Ashraf <[email protected]> Co-authored-by: Tobias Macey <[email protected]>
arslanashraf7
added a commit
that referenced
this pull request
Oct 29, 2021
* Canvas Commits (#223) * Sync canvas enrollments * Added instructor dashboard button to push edX grades to Canvas * Fixed canvas grade syncing JS * Changed 'edX' reference to 'MITx' * Use EDIT_COURSE_ACCESS permission instead of is_staff * Use OVERRIDE_GRADES rule instead * Use instructor_tasks for canvas work (#183) * Add per_page to paginated requests * Static method * Fix email lowercase mismatch * Use instructor_tasks to handle canvas work * Fix transaction error * Decorator needs to be at the top * Attempt to fix polling of tasks * More task bug fixes * Fix typo * Fix course key bug * Update message for push edx grades command * Use course_key so sync_canvas_enrollments uses the same message * Format the submitted time * Don't return output to prevent task output max size error (#189) Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]> * Reimplemented legacy dashboard enrollment data functionality (60a6ddc) * Fix syntax error in import statement * Cherry-Picked: Enrolment commits (#228) * Adding enrollment restriction for users based on social auth providers In order to prevent users outside of MIT creating a collaborator account and self-enrolling in courses we need to be able to restrict self-enrollment to users who authenticate using the MIT Kerberos SAML backend. This adds filtering of self-enrollment based on a feature flag that sets the allowed provider ids. (cherry picked from commit a4c53a1) * Bugfix for collaborator account access check Co-authored-by: Tobias Macey <[email protected]> * Fixing import path for canvas_integration plugin * Fix import namespace for relative imprts edX requires the full namespace for imports so importing e.g. `remote_gradebook.tasks` is no longer valid. Instead it requires `from lms.djangoapps.remote_gradebook import tasks`. * #229 change submission type and published state on canvas (#232) - Change submission type to "None" - Change published state to "False" * Fixing encoding of header row for posting to remote gradebook Posting to the remote gradebook via lmod proxy was failing due to a key error when parsing the CSV content. This was due to erroneously encoding a string object. This commit adds a check for the type of the header value to prevent double encoding it. * fix the imports * address feedback Co-authored-by: Asad Iqbal <[email protected]> Co-authored-by: George Schneeloch <[email protected]> Co-authored-by: Gavin Sidebottom <[email protected]> Co-authored-by: sar <[email protected]> Co-authored-by: Arslan Ashraf <[email protected]> Co-authored-by: Tobias Macey <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
per_pageto API calls using pagination, to increase the page size from 10 to 100