-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
docs: move application recipes to how-to guides #6693
docs: move application recipes to how-to guides #6693
Conversation
bad05ae
to
81b0fe9
Compare
81b0fe9
to
31bec51
Compare
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.
@mrmodise, thanks for the PR. Since this PR is in draft mode, I'm not sure if it's ready for review. I've put some comments on formatting-related items. Thanks.
Thanks @dhmlau for the comments. It will be ready this week. I faced issues pulling the latest master that caused some delays on completing this |
31bec51
to
020c4fa
Compare
020c4fa
to
cc56abe
Compare
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.
LGTM. I've reviewed mostly from the readers' point of view, and just had a minor comment. Please make sure you get other technical reviews before landing. Thanks @mrmodise!
979a6dd
to
429e4bb
Compare
@dhmlau agreed, I will wait for both @bajtos and @raymondfeng to review. Particularly, around the use of the Booter class. Amazing how there is so much hidden information in LoopBack website 😄. I have purposed to surface it |
@bajtos please review. Thanks |
Signed-off-by: mrmodise <[email protected]>
429e4bb
to
9ec06d8
Compare
Closes #6106
Checklist
npm test
passes on your machinepackages/cli
were updatedexamples/*
were updated👉 Check out how to submit a PR 👈