spring-petclinic-angular icon indicating copy to clipboard operation
spring-petclinic-angular copied to clipboard

Search not implemented

Open vfedoriv opened this issue 8 years ago • 10 comments

Need add search options (owner by name etc...)

vfedoriv avatar Feb 20 '17 21:02 vfedoriv

@vfedoriv @arey

Can i take up this issue

simrin051 avatar Sep 05 '21 08:09 simrin051

Hi @vfedoriv / @arey

I can take up this issue and form validation issue

simrin051 avatar Sep 17 '21 15:09 simrin051

Hi @vfedoriv / @arey

I am not clear with this issue. May i know whether search by owner name option is needed in the drop down box of owners.

image

simrin051 avatar Sep 18 '21 12:09 simrin051

I propose to rename the All entry menu to Search then add a filter input text in this screen. I don't remember if the REST API expose a search endpoint?

arey avatar Sep 18 '21 17:09 arey

Hi @arey There is an API exposed with last name. We can use this API.. @RequestMapping(value = "/*/lastname/{lastName}", method = RequestMethod.GET, produces = "application/json") public ResponseEntity<Collection<Owner>> getOwnersList(@PathVariable("lastName") String ownerLastName) {.....}

May i know what should be returned,should it return only owner name or owner details also

simrin051 avatar Sep 18 '21 17:09 simrin051

You may test the seach feature of the original version of Petclinic here: https://spring-petclinic-community.herokuapp.com/owners?lastName=E

arey avatar Sep 18 '21 17:09 arey

image image May i know is this fine @arey

simrin051 avatar Sep 21 '21 11:09 simrin051

Yes it looks like great

arey avatar Sep 21 '21 16:09 arey

May i know whether i should implement for Vets also? getAllVets() is the only method exposed API end point ( http://localhost:9966/vets.html ) Can you please advice @arey

simrin051 avatar Sep 21 '21 16:09 simrin051

Sorry @arey you have commented this on the wrong window. I have the below doubt.

May i know whether i should implement for Vets also? getAllVets() is the only method exposed API end point ( http://localhost:9966/vets.html ) Can you please advice @arey

simrin051 avatar Sep 21 '21 17:09 simrin051