Skip to content
This repository has been archived by the owner on Apr 19, 2021. It is now read-only.

Edits for issue #22 #24

Open
wants to merge 1 commit into
base: prod
Choose a base branch
from
Open

Edits for issue #22 #24

wants to merge 1 commit into from

Conversation

sishida
Copy link

@sishida sishida commented Oct 27, 2017

Edited the section, "Creating the Inventory Service with JAX-RS."
I added the following three pieces of text:

  1. Create an empty file with the following file name:
  2. Next, paste the following code into the empty file:
  3. Save the file.

Edited the section, "Creating the Inventory Service with JAX-RS."
I added the following three pieces of text:
1. Create an empty file with the following file name:
2. Next, paste the following code into the empty file:
3. Save the file.
@yeekangc
Copy link
Member

Thank you, @sishida. I am good with your proposed updates.

@lauracowen, do you think we only need to do this for the first step? Or, we should do it for say the initial couple?

@lauracowen
Copy link
Member

Is it okay if I hold off reviewing this until we've addressed a similar problem (but hopefully less severe) in the REST guide? OpenLiberty/guide-rest-intro#10. We spent more time reviewing the REST guide for usability and I think we should get it right and then use it as best practice for the others.

@yeekangc
Copy link
Member

yeekangc commented Nov 1, 2017

Yes, we should do that first. Thank you, @lauracowen.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants