-
Notifications
You must be signed in to change notification settings - Fork 24
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-42573) Update transactions in Functions info about read/write settings #860
Conversation
…ransaction session
✅ Deploy Preview for app-services ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Awesome job! 👏 Thanks for not just fixing the obvious issue requested in the ticket, but also the other problems you spotted while working on the page. Great work 😄
@@ -1,10 +1,10 @@ | |||
exports = function () { | |||
const client = context.services.get("mongodb-atlas"); | |||
|
|||
db = client.db("exampleDatabase"); | |||
const db = client.db("exampleDatabase"); |
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.
Well-spotted! Nice fixups - thank you! 🙏
@@ -439,7 +431,7 @@ To perform a transaction: | |||
} | |||
|
|||
The following example creates two users, "henry" and "michelle", and | |||
a uses a transaction to move "browniePoints" between those users | |||
uses a transaction to move "browniePoints" between those users |
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.
Ooo, another good spot - nice work!
Removing any mentions + code of creating a configuration object that sets user's preferred read/write settings and passing it to the session method. These settings are not configured in this context and not respected.
Pull Request Info
Jira ticket: https://jira.mongodb.org/browse/DOCSP-42573
Staging Links
Reminder Checklist
Before merging your PR, make sure to check a few things.
Release Notes
Review Guidelines
REVIEWING.md