Open sikai00 opened 1 year ago
No details provided by team.
[The team marked this bug as a duplicate of the following bug]
Unable to change the role of the prof with the role command
Using the example given in the app, in screenshot 2, we see that in screenshot 1, the roles of the professor remain as unassigned. This might be a UI bug but it causes a lot of problems especially when people want to look up for the roles of the prof as see it as unassigned. As such, I have given this a medium severity as looking up information is one of the basic features of this app should accomplish properly and that many people will be checking this information out. In addition, in the event that only 1 role can be assigned, an error message should be thrown instead of the command being accepted.
[original: nus-cs2103-AY2223S1/pe-interim#3649] [original labels: severity.Medium type.FunctionalityBug]
[This is the team's response to the above 'original' bug]
Thank you for pointing this out. Yes, in fact only 1 role can be assigned, and not multiple. This is stated in the UG
However, we recognise that we have outputted an incorrect error message here. We, however disagree that it is of medium severity. A user that has read the user guide will not try to input multiple roles for professors, as such it will only cause minor inconveniences in rare situations and will not affect normal operations of the product
Items for the Tester to Verify
:question: Issue duplicate status
Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)
Reason for disagreement: [replace this with your explanation]
says you can indicate multiple roles with comma
roles 2 roles/Coordinator, roles/Unassigned
was used.Steps to reproduce:
Currently has charlotte as unassigned role
After the command, Charlotte still only has 1 role, Unassigned.