Add mandatory id and annotations

[dhernandez]

Adding an id to Book entity class and with the annotations included the example works from scratch including OpenAPI docs.

[GregoireHebert]

can you remove the trailing space?

[dhernandez]

Fixed in d094c19

[GregoireHebert]

The example is needs also a @ApiProperty(identifier=true)
And to be honest, I should also have a property called $code
So the InputDataTransformer actually transform the isbn into a code.
And then in the OutputDataTransformer, from this code into the name.

[dhernandez]

The example only needs the annotation @ApiProperty(identifier=true) if you use another name to the property, like $code. But the example works without the annotation with the $id property.
Although, IMHO, Book entity shouldn't need a mandatory id property, because you have a BookInput and a BookOutput DTOs and BookInput have an id (called isbn).
So, to make a simple full example, that works with this issue, I think that adding an id called $id to Book is the clearest way.
Maybe, we can add also a note explaining the option of using $code property, annotated as identifier, and transforming it to isbn.

[GregoireHebert]

yup it's a good idea, at least to clarify the process.

[dhernandez]

You can check fabb572 🙂

[dhernandez]

Hi! What's happened with the PR? This is my first contribution so I don't know if you need anything else to merge. Please, tell me.

[alanpoulain]

Thank you @dhernandez.