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

Need for assignments requirment bug #8

Open
johnbenedictyan opened this issue Nov 11, 2022 · 1 comment
Open

Need for assignments requirment bug #8

johnbenedictyan opened this issue Nov 11, 2022 · 1 comment

Comments

@johnbenedictyan
Copy link
Owner

johnbenedictyan commented Nov 11, 2022

Bug Report:

What Happened:

  • Since the purpose of the application is a desktop app made for teachers, teaching assistants(TA) and professors for managing contacts of each other. There could be instances whereby the module has no assignments.
  • As such the requirement for at least one assignment whose weightage must be equal to 100% could be considered overzealous input validation
@nus-se-script
Copy link

nus-se-script commented Nov 14, 2022

Team's Response

The addAssignments command can be used to add other assessments such as Midterms, Finals, Projects, etc. I don't think there exists a module that does not have any graded components, because in that case, there will be no way for the student to receive a grade for the module.

Also there is absolutely no need to add any assignments. ABSOLUTELY REQUIRED fields are name, address, phone, etc... Basically the fields inputted when adding a person

This was an actual consideration by our team, whether or not to make assignment adding compulsory. We decided against it and therefore the addAssignments command is separate. Assignments can be added (or not added) to students after the students are added.

Therefore, we don't see this as a bug at all.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: [replace this with your explanation]


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

2 participants