-
-
Notifications
You must be signed in to change notification settings - Fork 194
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
transaction write update blocks #718
Conversation
… field or adding to or deleting from a set
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #718 +/- ##
==========================================
+ Coverage 90.89% 91.08% +0.18%
==========================================
Files 71 71
Lines 3414 3453 +39
==========================================
+ Hits 3103 3145 +42
+ Misses 311 308 -3 ☔ View full report in Codecov by Sentry. |
@andrykonchin Here is another transaction write PR. Sorry if I'm flooding you! Feel free to delay if you want to release the other stuff first. |
@andrykonchin This improves the transactions that got merged a few weeks ago. |
… to nil, removed documentation for upsert blocks
@andrykonchin I added the REMOVE and removed the docs for upsert blocks. I believe this covers all the feedback but please let me know if there's anything else. Thanks! |
…cks such as upsert{add()}
@andrykonchin I clarified some contexts and removed some docs and tests for unsupported functionality. Let me know if I missed anything. |
I will look at the changes a bit later. |
@andrykonchin Just updated the PR with those issues. |
Looks good. Thank you! |
transaction write supports blocks for updates to allow incrementing a field or adding to or deleting from a set