Open theeugenechong opened 2 years ago
May cause minor inconvenience for the user.
Team chose [severity.Low
]
Originally [severity.Medium
]
Reason for disagreement: I don't think this is a minor inconvenience because users who are already so used to the way edit_class
and edit_assessment
work (allowing optional parameters) may face this problem occasionally when trying to use edit_student
in the same manner.
Furthermore, realistically speaking, there is no reason to disallow a specific student parameter to be edited, hence this makes it less intuitive for the user.
The edit student feature requires that all the parameters are entered when editing a student's info. This may be quite inconvenient for the user who may only want to edit a specific field (e.g. student number).
It could have been implemented similar to edit class, edit assessment where the parameters are optional.