bryanwee023 / pe Goto Github PK

Should be using a., b., etc. instead for clarity.

The numbering then becomes inconsistent at step 6.8 where 1a. and 1b. is used.

From what I found, UG states that quantities are rounded off to 3 d.p., but the example here rounds of to the nearest 10.

There seems to be missing extensions that are important functional requirements of add ingredient and edit ingredient commands.

The extension that handles creation of duplicates seems to be missing. Even thought the parameters are valid (.e.g valid date and index), the ingredient will not be created if it creates a duplicate in the list.

When inputting a deliberately high index, doesn't tell me that index doesn't exist. Not that concerning since this is deliberate sabotage.

This is definitely a nitpick. When an item expires in a day, I would think it'll expire tomorrow.
Hence, Milk (below) should expire in 7 days (today's the 12th). but it is not flagged as expiring.

UG states that it'll flag within 7 days from today's date.

I don't think that NFR#5 is really a requirement, as it seems too stringent. For example, Fridgy doesn't support keyboard scrolling of opened recipes. Either NFR#5 should be removed or development should stick closer to the requirement.

No details provided by bug reporter.

Recipe card not dependent on Model. But should be dependent on recipe in model

When I add a flag -df into the programme, it registers it as a d flag, and assumes "f" to be part of the description. I think it would be more appropriate if the useris flagged with an error instead.

This isn't true as the app cannot accept non-English characters. Not really an issue since the English application is targeted towards English speaking users. But might want to clarify this for bilingual users.

To create a long recipe with many steps is difficult as the user is only provided a single lined command box to write paragraphs.

If he or she makes a typo, the user must retype the entire step list. There is no option to edit specific steps or append steps. This makes the recipe steps feature cumbersome to use. I would think the user would rather use a notepad.

Conversion of 0.001kg to 1g is supported but not 0.001l to 1ml. My 0.001l is converted to 0 instead.

Supported here

Not supported here

Similarly, other down-conversion of units are missing (g to mg). Not sure if this is a design choice or a oversight.

I'm confused as to what the compulsory column in Section 4.2 signifies.
For example -n is not always compulsory in commands (e.g. edit ingredient command)