ay2223s1-cs2113-f11-2 / tp Goto Github PK
View Code? Open in Web Editor NEWThis project forked from nus-cs2113-ay2223s1/tp
This project forked from nus-cs2113-ay2223s1/tp
appointment view
seems to list appointments with the wrong index number, starting from 2, unless they have some other meaning, which is not specified in the UG. This is causing severe failure in other features.
I believe the failure of indexing occurs when add appointment using appointment add
Change indexing logic to ensure they always start from 1
Labels: severity.High
type.FeatureFlaw
original: Brominne/ped#1
cancel a customer’s appointment.
When I add a pet with no name bue a space, your app still recognizes it and add to the pet list. I think name should not be empty
Labels: type.FunctionalityBug
severity.Low
original: bdthanh/ped#4
Add task failure, when following the example walkthrough of UG. I guess it can be due to being unable to find appointment #1
due to the wrong indexing of the appointment.
Rectifying the indexing issue in issue #1
could resolve this problem
Labels: severity.High
type.FunctionalityBug
original: Brominne/ped#3
Add some probability to the pats status like after each appointment and each task. Make it relative to species.
Issue
After I attempted to remove the first employee from the employee list, when I add a new employee I am expected to see the new employee be given an index number of 1. However, the program gave an index number 2 to the new employee instead.
Suggestion
Maybe generate the index number dynamically when listing.
Output
Labels: severity.Medium
type.FunctionalityBug
original: chydarren/ped#6
Issue
As seen below, when I tried to add a pet with only its health, the program does not catch the error in tags given and cause me to exit unexpectedly.
Suggestion
Suggestion is to check for all present tags.
Output
Labels: severity.High
type.FunctionalityBug
original: chydarren/ped#2
Add the durability of the service and make it decrease after each task.(can make it 100% first then no need to change the interface with other classes)
When I add a pet with no health value, the error is not handled properly causing the app to be crashed. Moreover, you do not have storage feature, then user will lose all of his/her data
Labels: type.FunctionalityBug
severity.High
original: bdthanh/ped#3
in case the owner wants to drop the membership
keep track of new pets
select corresponding service when entering an appointment
know whether the pets are recovered
keep track of the clinic’s schedule.
Issue
As shown below, attempt to remove the pet without specifying its index will cause me to exit unexpectedly.
Suggestion
Suggest to check for the availability of the parameters after checking for the presence of tag, and disallow all empty parameters to proceed.
Output
Labels: severity.High
type.FunctionalityBug
original: chydarren/ped#3
Issue
I am unsure if this was the intended behavior for the program that the team would like to keep, but I suggest to check for just alphabets in the name, as its quite rare that names have special symbols and numbers alone. That said, I cannot rule out this possibility entirely.
Suggestion
Check for special symbols and numbers before adding the employee name. Restrict to certain characters rather than allowing all kinds of input.
Output
Labels: severity.VeryLow
type.FeatureFlaw
original: chydarren/ped#4
Issue
As seen below, when you attempt to add the second pet with health that is invalid, the program does not inform that the health is invalid. The pet is not added into the list but the user is not informed.
Suggestion
Return an error message if any of the tags or parameters are invalid, and inform the user that the desired action was not performed, for the avoidance of doubt.
Output
Labels: type.FunctionalityBug
severity.Low
original: chydarren/ped#1
I can schedule a customer’s appointment.
Issue
As depicted below, adding an employee using duplicated name tags will result in the duplicated tag and parameter to be entered into the list as part of the first employee's name.
Suggestion
Suggest to check for any presence of duplicated tags.
Output
Labels: severity.VeryLow
type.FeatureFlaw
original: chydarren/ped#5
check all of the tasks assigned to a staff
newly hired staff can find their place.
i.e. when the pet add n/Eliza s/cat h/-1 is keyed, no feedback display is shown to the user or acceptance or error. However, under pet view, it is not recorded so not a big error. Can consider adding an error to notify the user that negative numbers are not allowed
Similar to the previous small bug for string length more than 1, because health should be input as 0 and 1 only as seen in the user guide. Would be good to create that restriction to prevent unknown feedback display.
Labels: type.FeatureFlaw
severity.Low
original: ivanthengwr/ped#1
to check all of the pets
When I input an unknown part to your pet add command, it is supposed to print out a error message but I got nothing. At least the app does not crash
Labels: type.FunctionalityBug
severity.Low
original: bdthanh/ped#2
An obvious faulty date of appointment can be recorded in the system.
From further testing, Any input string after d/
can be accepted
To improve, follow these steps:
DD/MM/YYYY
day
and month
is valid for the specific year-d/date
is not validLabels: severity.Medium
type.FeatureFlaw
original: Brominne/ped#2
When health is unrecognizable, pet add does not print exception or any messages. Therefore user won't know what error is to fix.
Labels: type.FunctionalityBug
severity.Medium
original: bdthanh/ped#1
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.