New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

add getOwnersPet implementation #119

Merged

arey merged 1 commit into spring-petclinic:master from alexandre-touret:fix/get_owner_pets_implementation

Jul 14, 2023

Contributor

alexandre-touret commented Jul 13, 2023 •

edited

Loading

Purpose

Fix #105

Add an implementation of OwnersRestController#getOwnersPet()

How is it done?

Adding an implementation in the OwnersRestController class
Adding two integration tests in OwnersRestControllerTests class

Validation

Running integration tests


          add getOwnersPet implementation

Contributor Author

alexandre-touret commented Jul 13, 2023

Hi @arey
Can you check it please?

arey added the enhancement label

arey merged commit adfe18b into spring-petclinic:master

1 check passed

Member

arey commented Jul 14, 2023

Thanks @alexandre-touret :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels