-
Notifications
You must be signed in to change notification settings - Fork 89
DOCSP-36298 : Refactor RQL for JSv12 #3266
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
DOCSP-36298 : Refactor RQL for JSv12 #3266
Conversation
c0ca369
to
41129af
Compare
Readability for Commit Hash: b28e7ac You can see any previous Readability scores (if they exist) by looking Readability scores for changed documents:
For Grade Level, aim for 8 or below. For Reading Ease scores, aim for 60 or above:
For help improving readability, try Hemingway App. |
c86e869
to
0183b3b
Compare
b905b74
to
ccd9cd4
Compare
ccd9cd4
to
e4f9c8f
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.
Wow, great job! This is a massive unit of work and I really like the new shape of the page. 🎉
I've got a handful of random comments, but only a couple are actually blocking - we should use the cpp-sdk
ID instead of cpp
, and there was a nill
typo I pointed out. The rest is pretty subjective, so feel free to take it or leave it, or move some of the comments out to a future unit of work.
9cd2167
into
mongodb:feature-consolidated-sdk-docs
✨ Staging URL: https://preview-mongodbmongodb.gatsbyjs.io/realm/feature-consolidated-sdk-docs/ 🪵 Logs |
Pull Request Info - SDK Docs Consolidation
Jira ticket: https://jira.mongodb.org/browse/DOCSP-3629
.ts
test fileJira ticket: https://jira.mongodb.org/browse/DOCSP-36618
Jira ticket: https://jira.mongodb.org/browse/DOCSP-36265
Staged Page
Page Source
PR Author Checklist
Before requesting a review for your PR, please check these items:
feature-consolidated-sdk-docs
branch instead ofmaster
Naming
.rst
files comply with the naming guidelinesLinks and Refs
[ ] Update any Kotlin API links to use the new Kotlin SDK rolesContent
[ ] API description sections have API details or a generic placeholder for all 9 languagesReviewer Checklist
As a reviewer, please check these items: