Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Support common helper methods like CreateIfNotExists #35

Closed
christopheranderson opened this issue Jul 9, 2018 · 3 comments
Closed

Support common helper methods like CreateIfNotExists #35

christopheranderson opened this issue Jul 9, 2018 · 3 comments
Assignees
@christopheranderson
Labels
feature Adds or changes functional behavior
Projects
Kanban
Milestone
RC1

Comments

@christopheranderson
Copy link
Contributor

There is a few customers who have requested or looked for CreateIfNotExists methods on our objects. In general, we've tried to stick to the simplest version of the SDK and want customer to write that logic themselves.

We should discuss whether it makes sense to support higher level methods that wrap underlying operations.
@christopheranderson christopheranderson added the discussion-wanted Need a discussion on an area label Jul 9, 2018
@christopheranderson christopheranderson added this to the Preview milestone Jul 9, 2018
@christopheranderson christopheranderson added this to Needs Discussion in Kanban Jul 9, 2018
@christopheranderson christopheranderson moved this from Needs Discussion to In Progress in Kanban Jul 16, 2018
@christopheranderson christopheranderson self-assigned this Jul 16, 2018
@christopheranderson christopheranderson added feature Adds or changes functional behavior and removed discussion-wanted Need a discussion on an area labels Jul 16, 2018
@christopheranderson
Copy link
Contributor Author

Some discussion was had of this. It turns out that C# does support this, so there's precedent for doing it already, and we have user feedback for wanting the method, so it seems like a no brainer.

@christopheranderson
Copy link
Contributor Author

We'll scope this issue to just "createIfNotExists" for all methods that have a create option.

@christopheranderson christopheranderson mentioned this issue Jul 23, 2018
Merged
christopheranderson pushed a commit that referenced this issue Jul 24, 2018
CreateIfNotExists (#35) (#66)
d4a3183 
* Add createIfNotExists for container/db

* Update to use new test method
@christopheranderson
Copy link
Contributor Author

Closed in #66

Kanban automation moved this from In Progress to Done Jul 24, 2018
christopheranderson pushed a commit that referenced this issue Jul 31, 2018
CreateIfNotExists (#35) (#66)
999e4b3 
* Add createIfNotExists for container/db

* Update to use new test method
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@christopheranderson christopheranderson

Labels
feature Adds or changes functional behavior
Projects
No open projects
Kanban
  
Done
Milestone
RC1
Development

No branches or pull requests
1 participant
@christopheranderson