You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Pulling this out from #675. I think this would be a useful enhancement.
A suggestion would be to expand the .update() command to avoid having to do 3 functions instead of 1. For example, this is currently the update() command:
In other words, these other new methods are added, and "changes" becomes "set" (but would still need to alias "changes" to "set" so it doesn't become a breaking change.) A person could use set, add, subtract together in a single call as above or just individually, examples:
This gives a simple pattern to convert into the actual sdk call. Since you've already used "increment" and "decrement" in other places, for conformity I guess it makes sense to use those instead of add/subtract, or at least alias them.
Pulling this out from #675. I think this would be a useful enhancement.
A suggestion would be to expand the .update() command to avoid having to do 3 functions instead of 1. For example, this is currently the update() command:
An option would be to use the commands set, add, subtract, etc so that you could issue update() this way:
In other words, these other new methods are added, and "changes" becomes "set" (but would still need to alias "changes" to "set" so it doesn't become a breaking change.) A person could use set, add, subtract together in a single call as above or just individually, examples:
This gives a simple pattern to convert into the actual sdk call. Since you've already used "increment" and "decrement" in other places, for conformity I guess it makes sense to use those instead of add/subtract, or at least alias them.
Originally posted by @ktwbc in #675 (comment)
The text was updated successfully, but these errors were encountered: