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

Umba Ilunga Jeannick 202025800049 & Mvondo Jean Claude Wilfried 20202… #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Jeannick94
Copy link
Contributor

@Jeannick94 Jeannick94 commented Jan 3, 2021

…5800038 Fixed bug No=74 unable to update attachment for posted assignments.
http://118.25.96.118/bugzilla/show_bug.cgi?id=74

Failed to update an attached file in a new assignment

I (as an instructor) posted a new assignment in the course (CSC322) - Software Project Management, titled ExamAnswerSubmission-Makeup (Individual).

I attached two files, StudentNumber-AnswerSheet.txt and ExamInstructions.txt.
I found an error in StudentNumber-AnswerSheet.txt, i.e., "ExamAnswerSubmission-ExamAnswerSubmission-Makeup" where "ExamAnswerSubmission" had been repeated twice.

Therefore, I tried to replace the old StudentNumber-AnswerSheet.txt with a new, correct one by clicking the Edit link and uploading a new StudentNumber-AnswerSheet.txt again. But I failed -- StudentNumber-AnswerSheet.txt remained the same.

…5800038 Fixed bug No=74 unable to update attachment for posted assignments.
@spm2020spring
Copy link
Member

@Jeannick94

  1. Potential file overwriting. If we store every attachment in the folder Lab_Report_Submissions, then what will happen if two attachments have the same file name?

  2. Your changes would create a new assignment after updating the old one, although the new assignment has the attachment file attached successfully.

  3. Could you sync with the master branch before you push? Check out this command: git pull origin master.

-Hui

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.

2 participants