-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
add context to Parse.Object.save #6626
Conversation
seems useful to me and looks good. curious any thoughts you might have @davimacedo and @dplewis if we merge, i can add to the PHP sdk |
@mtrezza The latest version of the SDK has been released and merged. You can update your branch and add the test cases. |
@dplewis Great! Let's see if the tests pass 🤞 |
Codecov Report
@@ Coverage Diff @@
## master #6626 +/- ##
=======================================
Coverage 93.88% 93.88%
=======================================
Files 169 169
Lines 11991 11994 +3
=======================================
+ Hits 11258 11261 +3
Misses 733 733
Continue to review full report at Codecov.
|
@acinader ready for review |
How can we use master key with context? Is this correct: |
Yes. It should work. |
Exposes the context object in Parse.Object.save() so that parameters can be made accessible in Cloud Code triggers beforeSave, afterSave.
Example:
Don't merge until