issues
search
ChenXy128
/
pe
0
stars
0
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
Could have mentioned that NIRC in read command is case-insensitive to add more clarity, as the wrong example shows upper case but the example shows all lower case.
#11
ChenXy128
opened
2 months ago
1
The quick starts says that all operating systems should download oracle java 11, but it is known that some MacOS is unable to run some fat jar files, hence resorting to Zulu java 11, but it is not mentioned here.
#10
ChenXy128
opened
2 months ago
1
The last NFR is not necessary met as to create a new person needs a long command in order to be a valid command for create, which can be quite cumbersome.
#9
ChenXy128
opened
2 months ago
1
To be accurate, there should be a white space in the argument of parse method in DeleteCommandParser class. Instead of parse("S.....X"), it should be parse(" S.....X") with a white space in the beginning
#8
ChenXy128
opened
2 months ago
1
There is still a mention of AddCommand in the Developer Guide, even though there is no AddCommand mentioned under Implementation section.
#7
ChenXy128
opened
2 months ago
1
multiple optional prefixes used can still be treated as correct format, even though the user guide did not mention this.
#6
ChenXy128
opened
2 months ago
1
since the user guide did not explicitly state that only 1 n/ is allowed, but when I n/ multiple times, only the value after the first n/ is read if there is no space between the second n/ and its value.
#5
ChenXy128
opened
2 months ago
1
Since country is specified to be optional, but cannot be empty, when I have the command "c/ c/<country>", this is allowed with an empty value for the first c/ to be empty, but not allowed if the second c/ is empty
#4
ChenXy128
opened
2 months ago
1
With an empty e/ followed by a e/ with inputs, I am still able to add the email of a person. When 2 e/ with values attached as command, only the latter email value is recorded.
#3
ChenXy128
opened
2 months ago
1
I am able to enter a wrong email format. When there is no ".com" or similar strings entered at the email, it still allows me to create this person with invalid email address.
#2
ChenXy128
opened
2 months ago
1
The error message is unclear. Since the command to create a person is very long, so it is quite normal to forget to add white space after every prefix, hence the error message should be more clear so I can find my mistake.
#1
ChenXy128
opened
2 months ago
1