jeminsieow / pe Goto Github PK

Person is not linked to Department or Office. Should also remove Address

module list cuts away and cuts back, when adding multiple contacts to a module which already has a tutor of a really long name

type the following

switch
[enter key]
switch
[enter key]

the UI does not change unless I click on the UI as well.

cannot see assigned contact to the module, when the module already has a contact with a really long name. In this example, cannot see Bernice Yu as a tutor

The instructor exists, just not in the current find list.
Steps to recreate:
find n/0
assign 1 m/[VV]

This behaviour of it required to be in the active list is not specified in user docs

The app is named FaculType, and the diagrams should reflect that as well. This may bring confusion to users

Index of contact disappears when name is really long

I really can't read anything on the sequence diagram. Font is way too small

Went into json file and changed instructor

It shows in the app, although this instructor does not exist

Here I removed the office to be an empty string.

This should be consistent. either change the deletePerson to deleteContact

Same email address for different people. 1 is Alex Yeoh, 1 is Alex Yeohj

If there are contacts of the same name, don't know which one is assigned to the mod

grammar error. should be 1 person listed

There's no use for priority section if everything is high priority. High priority features should make up the first iteration. I'm pretty sure the 1st iteration didnt have all these features.

This activity diagram does not bring a lot of value to the developer.

It says "2.FaculType shows a pop-up window."

Did not say what did the pop-up window contain. Too vague

long tags messes up UI, should add a limit

Cannot see full name if it is too long

The name of the json is addressbook. Could be confusing as the name of the app is FaculType

Same phone number for different contacts
1 is Alex Yeoh, 1 is Alex Yeohj, but same phone number

If the remark is really long, app starts to stutter when scrolling through

reset command description says "Resets FaculType to its initial state by clearing all entries of persons and modules"

but the initial state of FaculType has sample data. Could be more specific

Only the assign command has images of the app in use. This results in huge inconsistency.

I wouldn't know how the other command results look like

Too many examples here, and the explainations are confusing as well

Person is linked to Address. but in FaculType, Person does not contain Address, but contains Office and Department.