thesoggy / ped Goto Github PK

the blank code is not parsed

UG did not mention what is the actual format of the csv. It would be helpful if a user can see the headings that are used in a sample csv file

Perhaps a user should be prevented to rate multiple times in a day?

There is an additional space here as compared to

The requirements for department field can not be found in the UG.

Does not have position or department fields

Rating, once given, is unable to be edited

Int limit is actually $2^{31}-1$

ID gets from the table ID while id/ID gets from employeeId. Perhaps this can be standardised?

When taking leave non-business days should not be counted

$2^{32}-1$ is not parsed in the UG

Why not add n/NAME j/POSITION d/DEPARTMENT [p/PHONE] [e/EMAIL] [a/ADDRESS] [l/LEAVE] [t/TAG]…​ if in the command format the square bracket has been defined as an optional field?

Start date and duration headers are partially hidden, making it difficult for the user to guess what the headers are

Some commands rely on the table ID while some rely on employee ID so if the JSON file is directly edited the app will not update the table to reflect this.