-
Notifications
You must be signed in to change notification settings - Fork 85
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.
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
Proofread the tutorial #156
Proofread the tutorial #156
Conversation
I went through all the content and made the following changes: - fixed typos - reported some issues that were not that easy to fix (for instance the use of dead punkapi.com in some sections) - adjusted the first tutorials to use Jackson instead of JSON-B: we recommend using Jackson. Also it avoids having to go back to switch from JSON-B to Jackson when dealing with Spring - the reactive sections are still using JSON-B for two reasons: they are using some JSON-P features and I couldn't test things given the API is not available. When reworking them, we should switch them to Jackson too. This is a bit massive, I will go through the changes and add comments when they are needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kdubois Probably a good idea to add whitespace when reviewing. Small cog above the diff / Hide whitespace / Apply.
/cc @geoand @cescoffier
INSERT INTO Fruit(id,name,season) VALUES (7,'Watermelon','Summer'); | ||
INSERT INTO Fruit(id,name,season) VALUES (8,'Apple','Fall'); | ||
INSERT INTO Fruit(id,name,season) VALUES (9,'Pear','Fall'); | ||
ALTER SEQUENCE fruit_seq RESTART WITH 10; | ||
---- | ||
|
||
And append the following configuration in `application.properties`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't be needed but... there is a bug and it's still needed. I'll figure out what's going on and will adjust in a followup.
@kdubois once you agree on the changes, let me do a pass to check that I didn't break the formatting (unclosed backticks for instance). |
Looks great, thanks for going through and fixing / improving the content! I like the switch to Jackson as well. |
@kdubois OK, I went through the content again, and went through the all the steps I changed (so until REST Client). Things work but I had some bad surprises with dev mode when adding the extensions. Once, dev mode somehow didn't reload the model properly and I ended up with some classes (the MicroProfile REST Client ones) being missing. Even using It worked after a full restart of the dev mode. I'll try to reproduce this but it doesn't seem fully reproducible and is not related to my changes so I would say all these changes are good to go. |
@kdubois I added a small step to make sure people wait for Quarkus to be reloaded after having added an extension. Once PostgreSQL is installed, you have the Dev Service starting and it's relatively slow. I think that might explain what I saw (but we should try to see how we could fix it to survive the issue) |
@kdubois it would be nice to get this merged and published before next week as we are going to use this for the workshop at JCon. Thanks! |
LGTM! |
I went through all the content and made the following changes:
This is a bit massive, I will go through the changes and add comments when they are needed.