-
Notifications
You must be signed in to change notification settings - Fork 24
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
Update guide to use JSON-B instead of Jackson #25
Update guide to use JSON-B instead of Jackson #25
Conversation
README.adoc
Outdated
:page-duration: 20 minutes | ||
:page-releasedate: 2017-10-04 | ||
:page-duration: 25 minutes | ||
:page-releasedate: 2018-06-01 |
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.
Need to be updated to when the guide will be refreshed.
README.adoc
Outdated
:page-duration: 20 minutes | ||
:page-releasedate: 2017-10-04 | ||
:page-duration: 25 minutes | ||
:page-releasedate: 2018-06-01 | ||
:page-description: Learn how to consume a RESTful web service |
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.
Add "in Java using JSON-B and JSON-P" to the end.
//or custom constructor can be used | ||
this.title = title; | ||
this.artistName = artistName; | ||
this.totalTracks = totalTracks; |
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.
Alignment or extra whitespaces need to be fixed?
@JsonbProperty("albums") Album albums[]) { | ||
//or custom constructor can be used | ||
this.name = name; | ||
this.albums = albums; |
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.
Alignment or whitespaces to be fixed?
@@ -22,6 +22,7 @@ | |||
import javax.ws.rs.core.UriInfo; | |||
|
|||
import io.openliberty.guides.consumingrest.model.Artist; | |||
//import io.openliberty.guides.consumingrest.Consumer; |
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.
To be removed if not in use?
@KyleAure, looks like the latest changes are here including the fixes for #26. Kyle, can you please doubly confirm? @JunyiSun, can you doubly confirm that we have this PR tested and both the guide and code check out? Then, we can approve, merge and ask @kinueng to refresh the site for us. @evelinec, FYI. |
We are good to go. Thank you, @KyleAure! |
Remaining changes requested at the end of the previous pull request here: #20