You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I followed the User Guide by creating a new account with the following commands: register --user yourUsername --password AlexIsGreat --confirmPass AlexIsGreat --secretQn First Pet Name? --answer Koko
After sign-in, I tried to execute the same command with different confirmed password. As given below, register --user yourUsername --password AlexIsGreat --confirmPass AlexIsGrea --secretQn First Pet Name? --answer Koko
The application threw an error with the following message: Passwords do not match. Please try out again.
Even though the confirmed password doesn't match the password, I think this error shouldn't have a higher priority than the multiple account error. It makes more sense to throw an error indicating that user is not allowed to create multiple accounts.
This is a low FeatureFlaw because it does not hinder the ability of the user to register but only brings it slight inconvience.
However, this is part of planned enhancements #5 where we stated that currently there is no precedence of errors (ie. priority of errors) and we want to put errors of higher importance such as this first.
I followed the User Guide by creating a new account with the following commands:
register --user yourUsername --password AlexIsGreat --confirmPass AlexIsGreat --secretQn First Pet Name? --answer Koko
After sign-in, I tried to execute the same command with different confirmed password. As given below,

register --user yourUsername --password AlexIsGreat --confirmPass AlexIsGrea --secretQn First Pet Name? --answer Koko
The application threw an error with the following message:
Passwords do not match. Please try out again.
Even though the confirmed password doesn't match the password, I think this error shouldn't have a higher priority than the multiple account error. It makes more sense to throw an error indicating that user is not allowed to create multiple accounts.
[original: nus-cs2103-AY2324S1/pe-interim#6014] [original labels: type.FunctionalityBug severity.High]
The text was updated successfully, but these errors were encountered: