-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[Do Not Review] Docs feature branch: SQLite in DO #16903
base: production
Are you sure you want to change the base?
Conversation
Deploying cloudflare-docs with Cloudflare Pages
|
Files with changes (up to 15)
|
We're going to use
@harshil1712 Could you work on adding an example or even multiple? Kenton's example in the public blog should be available in docs as well. |
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.
4 files reviewed, 1 outstanding issue(s) found.
* Optional variable number of arguments that correspond to the `?` placeholders in `query`. | ||
|
||
#### Returns | ||
A cursor (`SqlStorageCursor`) that implements Javascript Iterator of query row results as objects with supported methods: |
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.
A cursor (`SqlStorageCursor`) that implements Javascript Iterator of query row results as objects with supported methods: | |
A cursor (`SqlStorageCursor`) that implements JavaScript Iterator of query row results as objects with supported methods: |
Issues:
- Style Guide - (Terms-error) Use 'JavaScript' instead of 'Javascript'.
Fix Explanation:
The change from 'Javascript' to 'JavaScript' aligns with standard spelling conventions and style guides. This correction is necessary to maintain the quality and accuracy of the documentation. No other issues were identified in the sentence, so only this term needs to be corrected.
TODO:
new_sqlite_classes