-
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: update todolist doc #1992
Conversation
0fa083a
to
6f0f392
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.
minor nitpicks, otherwise LGTM 👍
- create `todo-list.repository.ts` | ||
- update `index.ts` to export the newly created repository | ||
From inside the project folder, run the `lb4 repository` command to create a | ||
repository for the TodoList model using the `db` datasource. The `db` datasource |
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.
nitpick: TodoList -> TodoList
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 as far as I am concerned. Please address comments left by other reviewers before landing.
6f0f392
to
62e735b
Compare
Connect to #1908 and #1828.
Minor fixes in the documentation for the TodoList example, including adding
lb4 repository
command, and other miscellaneous updates.Checklist
npm test
passes on your machinepackages/cli
were updatedexamples/*
were updated