Skip to content

Fix for S3 Environment Error in Canvas-LMS Submission Reupload#2288

Open
imsujinpark wants to merge 2 commits into
instructure:masterfrom
imsujinpark:master
Open

Fix for S3 Environment Error in Canvas-LMS Submission Reupload#2288
imsujinpark wants to merge 2 commits into
instructure:masterfrom
imsujinpark:master

Conversation

@imsujinpark

Copy link
Copy Markdown

I recently encountered an issue (referenced here) related to the state "attached_temporary" in Canvas-LMS when reuploading submission zip files. The problem arises from the if statement unless @attachment.state == :unattached, causing errors during the reupload process in an S3 environment.

To address this, I've added attached_temporary state into the if statement.
With this adjustment, the code will no longer impede the reuploading of submission zip files in S3 environments, preventing errors specifically when "create_success" is encountered after uploading.

@CLAassistant

CLAassistant commented Nov 14, 2023

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


imsujinpark seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants