pe's People
pe's Issues
Description for notes about tags could be clearer
The UG states:
To indicate the priority level, simply follow the tag name by typing :p[1/2/3/4]
after the tag name.
It is not clear here that the priority level and tag name have to be separated by a space, and users only learn this when they see the example given below.
I was confused when I tried to enter t/important:p1
and p1 became part of the tag name until I saw the given example.
Numeric client names are allowed
Steps to reproduce:
Use the command add n/123456 p/98765432 e/[email protected] a/John street, block 123,#01-01, i/package1
Expected result:
Throw an error indicating that the client name should contain alphabetical numbers and not numbers
Actual result:
Numeric client names are allowed
Screenshot:
Undo limit
Numeric insurance package names are allowed
Steps to reproduce:
Use the command add n/John Doe p/98765432 e/[email protected] a/John street, block 123,#01-01, i/123456
Expected result:
Throw an error indicating that the insurance package name should contain some alphabetical characters, not just numbers
Actual result:
Numeric insurance package names are allowed
Screenshot:
Clear command is allowed even if client list is empty
Import does not validate whether or not there are duplicate client names
Steps to reproduce
- Use the
export
command to export the default client data - Open the csv file and edit the second client's name to match the first client (Alex Yeoh)
- Use the
import
command to import the same csv file back into the app - Use the
list
command to save the imported data
Expected result:
Throw an error to reject the imported data since there are two clients who share the same name, which would not have been allowed if the data was manually keyed in using the add
command
Actual result
Invalid data is allowed to be imported
Screenshot:
Can add tags of the same name and different priority level
Steps to reproduce:
Use the command add n/Jackson p/92341888 e/[email protected] a/laney street, block 123, #01-01, i/package1 t/important t/important :p1 t/important :p2
Expected result:
Could have any of the 3 following responses
- Throw an error indicating that a client cannot have more than 1 tag with a particular name
- Accept only the first input when there are duplicate tag inputs
- Accept only the last input when there are duplicate tag inputs
Actual result:
All duplicate tags are added to the client, which can be confusing as the client can have the same tag multiple times and all with different priority levels
Screenshot:
List command is allowed even if client list is empty
No legend for tag priority level within the app itself
Steps to reproduce:
-
Use the command
add n/Jason p/92341888 e/[email protected] a/laney street, block 123, #01-01, i/package1 t/important
-
Use the command
add n/Jackson p/92341888 e/[email protected] a/laney street, block 123, #01-01, i/package1 t/important :p1
Expected result:
UI should indicate to me which color corresponds to which priority level
Actual result:
No indication of what priority level each color represents, have to refer to UG. This can be frustrating for beginner users.
Screenshot:
Format for package related commands could be clearer
The formats provided under the Package related commands
section are as follows:
addp i/PACKAGE_NAME d/PACKAGE_DESC
editp i/PACKAGE_NAME d/PACKAGE_DESC
In every other section of the UG, the formats are all clearly spelled out in full. However, in this section, PACKAGE_DESCRIPTION
has been shortened to PACKAGE_DESC
, which can be confusing especially for beginner users as it is not immediately clear what DESC
is short for.
It would be good for clarity if PACKAGE_DESCRIPTION
was also spelled out in full like the rest of the UG.
Find command is allowed even if client list is empty
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. ๐๐๐
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google โค๏ธ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.