speedweener / pe Goto Github PK

Incorrect error message. Should suggest that there only one project has been added

I do think that the specified help commands are a quite redundant. After all, the user should understand how the help command works after the first example. The additional help portions make the UG longer then necessary

opt block should be used instead if there is not else portion

Minor issue where pressing the enter key with empty commands does not prompt an error message. This could cause confusion for users who do not know what mode the app is in.

I think there should be additional spacing between portions of the UG. Else, a lot of the sections blend together making it quite hard to distinguish the sections

Would be better if the app could detect if there were changes and print a different message. In this case, sound of to the user that the priority was already high

I think this may be a bug that arises from continously adding sprints of the same name. This causes it to be impossible to add other sprints even on different dates

Error message for invalid project id could be more clear, such as the range of project ids currently exisiting

Should be alt instead of opt for two separate conditions. Conditions should also be within square brackets []

It is not really clear from the user guide that there is a maximum number of sprints

Should use ref blocks to simplify

The arrows are quite messy, with some dotted lines turning into solid lines, and dotted lines emerging from soolid lines. Should also use labels to clearly depict the types of relationships in the diagram.

The box for Parser should also be of another colour/thickness to avoid confusion with the arrows

Fast typers who additionally add a space would trigger a missing parameter error which should not be the case

Rather than using markdown formatting, i think it would be better to include images of the app and expected output. Else, it becomes harder for the user to compare the guide with the app.

More images would also make the user guide more visual and appealing to read. At the moment I feel that there is a bit too much text.

Hyphen results in app regarding input as missing parameters. Perhaps return another error message?

Not really a lot of information from the screenshot, but after adding 3 sprints on invalid days, I was unable to test the other features as 1) could not add any more sprints and 2) could not delete current sprints
There is no section in the user guide on how to delete sprints either

Seems wrong to depict this as a class diagram, because it is simply showing the methods in Ui.

The arrows should also only be used to show association

Command suggested by app does not do anything. Missing space in prompt to user to be project/ list

the
Inheritance from abstract class Command should be shown with arrows pointed towards the superclass not the subclass
The speech bubble boxes are also a little confusing

Similar issue, i think a better error message is need for users who accidentally type in an extra hyphen

Based on the flow of the user guide, i think most users would see a number 2 over here instead. since another project had been already added in the setup