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

[BUG] If Assignment ID Changes on Canvas after Config File is Pulled, Posting Will Fail #41

Open
6 tasks
gregbell26 opened this issue Jan 17, 2023 · 0 comments
Labels
bug Something isn't working Module: Canvas This issue relates to the Canvas API Wrapper Module: Config This issue relates to the Config module Module: UI This issue relates to UI

Comments

@gregbell26
Copy link
Member

Description

Issue

If an assignment is recreated or moved, the ID on Canvas is not guaranteed to remain the same. Because of this, grading posting will fail with a 400 HTTP status code as the script has an outdated ID.

Steps to recreate

  1. Create an assignment on Canvas.
  2. Create a config file for the script.
  3. Delete and recreate the same assignment on Canvas.
  4. Attempt to post grades for that assignment.

Solution

During assignment selection, the assignment should be verified to be writable on Canvas. If it is not, ask the user what to do: ignore the assignment, or reenter the id (Override the error). This relates to #37 and should use components from that system.

Acceptance Criteria

  • Does not duplicate work from [ENHANCEMENT] Add support to create and override assignment mapping #37.
  • Assignments are verified to be writable during the assignment selection phase of the program.
  • User is able to choose an action to take from Ignore and Override
  • Ignore option removes the assignment from the list of assignments to be graded.
  • Override option allows the user to enter the ID of the assignment
  • Override option causes config file to be updated (May need to move to separate issue)
@gregbell26 gregbell26 added bug Something isn't working Module: UI This issue relates to UI Module: Canvas This issue relates to the Canvas API Wrapper Module: Config This issue relates to the Config module labels Jan 17, 2023
@github-project-automation github-project-automation bot moved this to New Feature in Grading Script Jan 17, 2023
@gregbell26 gregbell26 moved this from New Feature to Bug in Grading Script Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Module: Canvas This issue relates to the Canvas API Wrapper Module: Config This issue relates to the Config module Module: UI This issue relates to UI
Projects
None yet
Development

No branches or pull requests

1 participant