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

Allow assignments to be created and grades synced in same view #174

Open
gsidebo opened this issue Aug 25, 2020 · 5 comments
Open

Allow assignments to be created and grades synced in same view #174

gsidebo opened this issue Aug 25, 2020 · 5 comments

Comments

@gsidebo
Copy link

gsidebo commented Aug 25, 2020

#172 added functionality that adds edX assignments to Canvas and then pushes student grades for those assignments. Due to a minor bug, if an assignment is created as part of this view, the grades for that assignment aren't pushed in the same view. You need to click the button a second time to sync those grades. We want the grades to be pushed whether or not the assignment is newly created in the view.

Tech note: The view includes some logic that builds a dict of assignments that exist in Canvas. That dict is used to figure out which grades should be pushed. Basically we just need to add to that dict when new assignments are created.

┆Issue is synchronized with this Trello card by Unito

@gsidebo gsidebo added the bug label Aug 25, 2020
@pdpinch
Copy link
Member

pdpinch commented Sep 2, 2020

@gsidebo is this still an issue?

@pdpinch
Copy link
Member

pdpinch commented Sep 11, 2020

@noisecapella is this still an issue?

@shaidar
Copy link
Collaborator

shaidar commented Dec 4, 2020

@gsidebo Is this still a problem or has it already been resolved?

@gsidebo
Copy link
Author

gsidebo commented Dec 4, 2020

It hasn't been resolved yet (and sorry for missing the earlier messages). I'd consider this low priority but the issue is still there

@shaidar
Copy link
Collaborator

shaidar commented Jan 5, 2022

@pdpinch Not sure whether Gavin or someone else addressed this already but if not, does it need to reassigned or it's not a priority?

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