spring-projects / spring-petclinic

A sample Spring-based application

Apache License 2.0

7.79k stars 23.97k forks source link

New Feature - #1671 - Added Profile Page for Vets #1672

Closed mosbat closed 1 month ago

mosbat commented 2 months ago

1671

dsyer commented 1 month ago

The effort is appreciated, but I'm not sure we need this (and the usual policy on new features is to decline). Also the UUID change is kind of unnecessary as well as being invasive.

mosbat commented 1 month ago

The effort is appreciated, but I'm not sure we need this (and the usual policy on new features is to decline). Also the UUID change is kind of unnecessary as well as being invasive.

No worries, I didn't know about the policy :) but thanks for letting me know! I had some time to kill :).

You're right that the UUID change is not necessary; probably overkill.

If you like I can close this PR and work a bit on the existing code base with smaller changes :) .

dsyer commented 1 month ago

Contributions always gratefully accepted. Thank you again.