nus-cs2103-ay1920s1 / pe-dev-response Goto Github PK

If a command like "clear" is accidentally executed by the user, it can result in severe data loss with no way to recover the data. There is no peace-of-mind provided for the user.

Instead of listing all available commands, opens a window with the link to the UG. Behaviour is different than what is described.

Summary:
The tags in the budget section overlaps the date.

Steps to reproduce:

1. Open the application
2. Create a new expense category that is of sufficiently long length (in this case: Mmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmm)
3. Edit the existing budget by adding this new tag to the budget line.

Expected results:
The tag name should be truncated so that the date can still be visible clearly.

Actual results:
The full name of the tag is printed out and overlaps the date, causing the user to be unable to see the date for the budget.

Screenshot of issue:

Summary:
The user guide does not mention how we can change the sample data form. Changing it by adding a comment to the start of the json file, as layman users may do, causes the app to load incorrectly, despite the remaining of the json file being of valid format. However, the user guide doesn't specify how the json fie can be changed or what can't be changed.

Steps to reproduce:

1. Add a preamble to the mams.json file:

Expected results:
The app should still load correctly, since the user has not tampered with the actual data.

Actual results:
The app starts with an empty mams.json file and flags the file as incorrectly formatted.

Summary:
The user is unable to see all the information presented in the application with a small window.

Steps to reproduce:

1. Open the application as normal.
2. Resize the application to the smallest possible

Expected results:
The information presented by the application should be able to be resized so that all information can be seen clearly. Alternatively, there should be a scrollbar available for me to scroll to the right to see the contents that are hidden.

Actual results:
Some of the information, especially the ones on the right sidebar, is cut off and unable to be seen.

Screenshot of issue:

Summary:
The reject command doesn't allow "a/" prefixes in the optional remark, but nothing is mentioned that that isn't allowed.

Steps to reproduce:

1. type and run the following command:
   reject a/2 r/"the guy drew a a/ on the wall"

Expected results:
Appeal 2 is rejected with the input appeal remark.

Actual results:
The command is flagged as invalid.

Screenshot of issue:

I tried to execute the markAttendance command but unfortunately met with an error. I followed the instructions from the help message but turns out it is not according to the user guide. I keyed in as instructed numbers separated by comma but did not work. User guide did not document this too.

Summary:
When the description of the appoint message is too long, the message will be truncated on the UI (Cards in the appointment tab)

Steps to reproduce:

1. Using an existing appointment, type add-edit with a very wrong desc/
   e.g. appt-edit 4 desc/The post has just arrived and in it a very nice surprise, the discovery that Jacques Seguela, one-time adviser to President Mitterrand, now close confidant of President and Madame Sarkozy (indeed he intoduced them), and something of a legend in French political communications, has dedicated his latest book to little old moi.With apologies for the missing accents here and in the French bits of the long posting which follows – the dedication to ‘Le Pouvoir dans la Peau‘ (Power in the skin) reads ‘A Alastair Campbell, mon spin doctor prefere’ (three missing accents in one word – mes excuses sinceres).So what did I

Expected results:
The very long description should be displayed on the UI (Cards in the appointment tab).

Actual results:
The very long description is truncated on the UI (Cards in the appointment tab)
.

Screenshot of issue:

A vehicle created by a user cannot be deleted by the same user. All deletes are only allowed by admin.

I tried to key in an invalid module index, but the error message is not correct, and says that command is invalid when it is correct according to user guide.

When a wrong path is given in preferences.json, the app could not find the incidentManager.json as expected. However the file still exists. The app then proceeds to overwrite the old file with default data and thus I loose all data on my previous session.

Heres a screenshot of the wrong path i intentionally put in; note the s in incidentManagers.json

Sometimes, I try to click on the text input bar many times and it responds at random. i.e. at times when I click, it allows me to enter the text, but other times there is simply no response and no sign that any clickling allows me to enter the text.

I think there is a formatting error for one of the example commands, as highlighted.

Summary:
The setReminder command does not work as expected.

Steps to reproduce:

1. Open the application
2. Ensure that the reminder section is empty
3. Type a setReminder command (in this case: setReminder 1 typ/expense n/Mala p/1d f/daily)

Expected results:
The software should inform me that there are no reminders to set.

Actual results:
The software tells me that a new reminder has been added, but nothing appears to happen.

Screenshot of issue:

Summary:
Weird behaviour of addExpense command.

Steps to reproduce:

1. Open the application, add expenses until budget is exceeded (starting with extremely large numbers but within the allowable range)
2. Continue to add expenses (in this case: addExpense cat/Food n/Mala amt/1 d/2019 11 09 tg/food tg/indulgence)

Expected results:
I should be notified that I am successful in adding the expense and that I have exceeded the budget.

Actual results:
I am notified with an error that the budget amount should only contain numbers, and it should be only 2 decimal points at maximum. It should also not be bigger than 9999999.99, which is not the case since my expense is a legitimate number)

Screenshot of issue:

Waste List show thw wrong month during when glist list m/aug 2019 command is passed.

Steps to reproduce

1. Type glist list m/aug 2019.

Expected
List of food waste duiring August

Actual
List of food waste during July

Following the instructions for manual testing, I used "list" but the command is not recognised.

This command format is used to cover a variety of potential issues with commands.
For example, in this case, the user attempts to create a budget with an amount $0. The error message does not tell the user what is wrong in a way that is sufficiently helpful.

Another example:

Summary:
The command format for the editIncome command is incorrect.

Steps to reproduce:

1. Open the application.
2. Follow the editIncome command format provided in the User Guide. In this case, I used editIncome cat/Gifts n/selling mysoul amt/324 d/2019-11-02 by following the command format.

Expected results:
I should be able to edit the income based on the parameters given.

Actual results:
An invalid command format error is thrown.

Screenshot of issue:

I tried to add more than 1 nusid and matric numbers into the parameters. However, the error message given is not correctly describing the situation.

The error message says that the module/tutorial does not exist when it exists.

Initially specified with & in between parameters, but eventually shows examples without it.

edit-v command does not allow changing of vehicle number but allows changing of type. Inconsistent reasoning to not provide such feature.

EDIT:

it is found that the argument to edit vehicle number is vn but is not stated in format. Moreover argument used in add-v is vnum, inconsistent naming scheme.

Summary:
The user guide provides a toggle command example (which is "toggle ae`") and it does not work.

Steps to reproduce:

1. Open the application
2. Type in the command "toggle ae`" as shown in the example provided in the User Guide.

Expected results:
Something should happen.

Actual results:
An invalid command format error is given.

Screenshot of issue:

Summary:
When an invalid date is typed, e.g. 31-02-2019, the app accepts the date and round off the date to the nearest valid date

Steps to reproduce:

1. Using an existing patient, type appt-add p/2 sdt/31-02-2019 1300 edt/09-12-2019 1330

Expected results:

1. The app should reject the input and tell the user that the date is wrong.

Actual results:

1. The app accepts the date and round off the date to the nearest valid date

Screenshot of issue:

Summary:
The User Guide has extra commands in the command summary not mentioned anywhere else in the User Guide.

Steps to reproduce:

1. Open the User Guide
2. Navigate to the Command Summary section.

Expected results:
I should be presented with a summary of only the commands that are listed in the User Guide previously.

Actual results:
I am presented with additional commands that are not mentioned elsewhere in the User Guide. Some examples include addQuotaCondition, addClassCondition, addDateCondition, etc.

If the command is coming in v2.0, it should at least be mentioned briefly in the User Guide.

When a user created an account and is logged in, he is still unable to edit his credentials e.g. his name.

Few functions of the app are covered in the instructions for manual testing.

Section G.3. is also clearly incomplete.

![image.png](https://raw.githubusercontent.com/nus-cs2103-AY1920S1/pe/master/imageFolderTXT/0283915143_

This greatly reduces the usefulness of the budget feature to the user. It is interesting to note that expenses made during the budget's specified time period added after the budget is created will be automatically added, yet there is no way for the user to manually do so.

The incidence status gets cut off and is hardly visible on macOS.

Summary:
The app crashes and doesn't even run when we input an extremely high number of module availabilities, despite the user guide saying that the sample data "may be replaced with actual data differing by semester"

Steps to reproduce:

1. Change the module quota in mams.json to an large integer:

Expected results:
The app either informs the user it cannot handle such a high quota, or fails gracefully

Actual results:
The application throws an error and doesn't even start up.

Screenshot of issue:

If the command is login u/Agent01 w/alksndalksjd w/password it still accepts the login. This behaviour is not specified in the UG.

Summary:
The userguide specifies that the json input data can be replaced with actual data. Upon replacing the app with other data, the app is unable to load.

Steps to reproduce:

1. Replace the entries in mams.json with the following:

2. Run the app using java -jar [the app name].jar

Actual results:
The following error is thrown:

Expected results:
MAMS should inform the user that the data provided is in incorrect format, instead of silently failing.

Screenshot of issue:

I tried to key in the command addStudent in quick start but it did not work as expected.

Summary:
The UG says "Weme will not add text that would exceed the image boundary. If it detects such a scenario, it will print an error message." But no error is thrown when a long text is added that exceeds the image boundary.

Steps to reproduce:

1. Add a long text > 50 characters

Expected results:
Error to be thrown

Actual results:
App allows it.

Screenshot of issue:

mergebought still can be undone despite UserGuide saying otherwise

Steps to reproduce:

1. set up the shopping list
2. type slist bought 2 e/20/11/2019 a/UNITS
3. type slist mergebought
4. type undo

Expected:
Error message which mention that undo command cannot be done

Actual:
Undo still can be done

Command is still able to go through when the wrong prefixes are used for the format.

Steps to reproduce:

1. Type wlist report s/Mar 2019

Expected:

1. Invalid command result as the prefixes are wrong

Actual:

1. Graph is still shown and processed

Again, while this is the behaviour intended by the team, it does not make sense from a user's point of view. What if I want to concurrently track my daily expenses while saving for a trip?
This is likely to be a common use case and it is a shame that the app cannot handle it.

Similar to previous issue, user is unable to delete the account he created himself and is logged in.

The window can be resized beyond reasonable limits, causing much content to be cut off.

At the minimum size of the window, only 1 column is in view. Yet this visible column is the "left" column by default, which shows all expenses, and does not react to user commands.
While this is indeed documented in the UG, it seems like having the "centre" column visible by default/when the window is small would be more useful to the user.

For example, in the attached screenshot, I have entered the command "view 1" expecting to see my budget for the Japan trip but it is not visible in my current view. The app's behaviour may differ from the user's expected behaviour, causing confusion.

JSON with extra properties are accepted and not validated as malformed json files. In this example i added a z component to preferences.

When description of a meme is too long, the meme is not shown in the window when the 'view' command is used. Cannot use horizontal scroll function to see the meme as well.

Steps to reproduce:

1. Add/edit a meme's description to be very long >100 characters.
2. View the edited meme.

Expected results:
Still be able to see the meme. Or scroll to see the meme.

Actual results:

In the user guide, it states that the student must be identified by a combination of parameters as followed:

However, I cannot find the parameter input for the nusid and matric number in the format of the command.

I understand that it is an optional parameter, but it should be in the general format and have listed as a optional field instead of being dropped.

As mentioned in previous issue, similarly in summary of commands, edit does not specify that it can change vehicle number.

Summary:
The addReminder command does not seem to be working.

Steps to reproduce:

1. Open the application, ensure that the Reminders section is empty.
2. Use the addReminder command (in this case: addReminder n/Don’t be broke. l.b/10 u.b/20 start/2019/01/01 end/2019/12/30)

Expected results:
A reminder should be added and appears under the Reminders section.

Actual results:
The software says that a reminder has been successfully added, but it is not reflected in the Reminders section.

Screenshot of issue:

In the user guide, some command parameters have square brackets enclosing it but, not others. It is not entirely clear to me why the square brackets are there.

With square brackets:

Without square brackets:

No explanation given (or found):

pdf format did not show the ticks and crosses of the displaying food wastage report in UserGuide

Expected

Actual

Summary:
Chart does not show when there is 0 data.
Labels for the data are overlapping as well

Steps to reproduce:

1. Like/dislike memes such that it makes a section of the pie chart as small as possible.

Expected results:
Labels are still shown for each category

Actual results:
Only some labels are shown

Screenshot of issue:

No error was shown when a wrong prefixes is entered in the command

Steps to reproduce

1. type wlist list r/aug 2019

Expected

1. An error message to show that invalid command format is entered.

Actual

1. Command box did not display any error result. The previous command message is shown instead.
2. Command is unable to enter and parsed.

addmod s/12 m/

After list is sort by alphabetical, items added are not automatically sorted

Steps to reproduce:

1. Run glist sort by/alphabetical
2. Type glist add n/Apple Pie a/300g e/30/11/2019 t/healthy t/fruit

Expected:
Apple Pie should appear 2nd in the list

Actual:
Apple Pie appear last in the lsit