Antoine Rey

@firasrg for my point of view, the aim of this demo application is to explain to junior React developers how to build a React application using best practices and without...

Hi @firasrg, thank you for taking into account all the changes. I will let @nilshartmann review the ReactJS updates in detail.

The `equals` method is not implemented in the `Owner` entity. Instead of I propose to use the `getId()` for comparing both instances : ``` if (!pet.getOwner().getId().equals(owner.getId())) ```

To fix this issue, we can start by adding a failed test case to the `OwnerRestControllerTests`. Then, you're right: I'm not sure we need to call the `findPetById` method. The...

@firasrg I let you review my fix in the PR [bad request on GET /api/owners/{id}/pets/{pe…](https://github.com/arey/spring-petclinic-rest/commit/29a3295a4bbd74b939663d8ff94db3059859bcdc)](https://github.com/spring-petclinic/spring-petclinic-rest/pull/150)

Good catch @Isaac-Andradee: > The

Hi @firasrg Thanks for your proposition. We are looking contributors to upgrade all the forks of the Petclinic Community and you are welcome. I think you can indeed draw inspiration...

Hello Nils and Firas. If we could use the existing [spring-petclinic-rest](https://github.com/spring-petclinic/spring-petclinic-rest) as backend it coud be great. The [spring-petclinic-reactive](https://github.com/spring-petclinic/spring-petclinic-reactive) demonstrates how to use a git submodule.

Hi @firasrg We don't yet have a mobile version of Petclinic dedicated to smartphones. The Kotlin version is a classic webapp, not an Android application. Having a fork of Petclinic...

I've got the same error on my M4: ``` # A fatal error has been detected by the Java Runtime Environment: # # SIGILL (0x4) at pc=0x0000ffff8733fc5c, pid=1, tid=7 #...