Add Pet - Githubissues

BekahT / VetApp

A Java desktop application to be used in a veterinarian's office for managing clients, pets, and appointments.

0 stars 0 forks source link

Add Pet #10

Closed BekahT closed 4 years ago

BekahT commented 4 years ago

Staff can add a new pet to an existing client with pet name, species, gender, and date of birth.

BekahT commented 4 years ago

Redirects to Clients tab, find client, and then add pet.
Species dropdown (cat, dog, guinea pig, fish, rabbit, ferret, etc.)
Gender dropdown
DOB use datepicker
DOB must be in the past. Cannot be in the future.

BekahT commented 4 years ago

Pet test data for Angela: https://theoffice.fandom.com/wiki/Angela%27s_cats

BekahT commented 4 years ago

[x] There's a maskformatter on petdob, but it's not displaying/auto-formatting as you type. It should show the dashes and auto-format for the user.
I shortened some error messages so they would fit in the window. We should either make the window wider, or keep our errors short.
[x] The buttons for Submit and Cancel have black text, while the Add client ones have blue text. Pick a color and use it for all submit/cancel buttons.

cp-umuc commented 4 years ago

I don't like saying click the paw button - sounds weird. The User Guide should explain that it is the Add Pet button so we should call it that. If you want to keep the paw part then maybe "Paw icon" or something?

cp-umuc commented 4 years ago

After we add a new pet, we should be returned to the Pets dashboard instead of the Clients dashboard. I know you mentioned returning to the Clients dashboard when canceling the Add Pet and that's fine - like in case they clicked on the wrong client - but if they are successful they should be taken to the Pets dashboard with the updated Pet list.

cp-umuc commented 4 years ago

No page heading on the Add Pet page like is seen on the Add Client Page.

cp-umuc commented 4 years ago

Invalid pet name results in error message going off the screen.

cp-umuc commented 4 years ago

Same cut off for date of birth in the future error.

cp-umuc commented 4 years ago

No error message displayed for empty date of birth. (Nothing happens when you click Submit.)

BekahT commented 4 years ago

I resolved the no error on empty dob one, in my update to the edit pet merge request.

BekahT commented 4 years ago

I don't like saying click the paw button - sounds weird. The User Guide should explain that it is the Add Pet button so we should call it that. If you want to keep the paw part then maybe "Paw icon" or something?

Maybe something like "Please select a client and click the Add Pet button (paw icon) to create a new pet."

BekahT commented 4 years ago

Shrink error font to match size used in Add Client.

BekahT commented 4 years ago

Changed all warnings to font size 14 from 18 (also matches add client warning fields). Updated Edit Pet as well.
Updated Add Pet Instructions message.
Added title to Add Pet window.
Redirect client to pets tab upon successful pet creation