Update PersonResource.java - Githubissues

quarkusio / quarkus-quickstarts

Quarkus quickstart code

https://quarkus.io

Apache License 2.0

1.97k stars 1.47k forks source link

Update PersonResource.java #1434

Open jjpinto opened 4 months ago

jjpinto commented 4 months ago

update should accept person entity only since it is not using the id parameter.

Check list:

Your pull request:

[ ] targets the development branch
[ ] uses the 999-SNAPSHOT version of Quarkus
[ ] has tests (mvn clean test)
[ ] works in native (mvn clean package -Pnative)
[ ] has integration/native tests (mvn clean verify -Pnative)
[ ] makes sure the associated guide must not be updated
[ ] links the guide update pull request (if needed)
[ ] updates or creates the README.md file (with build and run instructions)
[ ] for new quickstart, is located in the directory component-quickstart
[ ] for new quickstart, is added to the root pom.xml and README.md

jjpinto commented 4 months ago

The update function was reviewed. If the parameter Id is passed then it should take an action.