feat(update): remove create or update
create or update introduces a new pattern in addition to the existing flow of:
- getting a resource.
- if NOT_FOUND, send a create.
- if found, send an update.
In the future, this could be re-added as an Apply method, which would map to the appropriate REST method of PUT.
🍱 Types of changes
What types of changes does your code introduce to AEP? Put an x in the boxes
that apply
- [x] Enhancement
- [ ] New proposal
- [ ] Migrated from google.aip.dev
- [ ] Chore / Quick Fix
📋 Your checklist for this pull request
Please review the AEP Style and Guidance for contributing to this repository.
General
- [x] Basic Guidance is met.
- [x] Ensure that your PR references AEPs correctly.
- [x] My code has been formatted
(usually
prettier -w .)
💝 Thank you!
I'm in favor of this as long as the goal is to add an Apply method. If we don't add Apply long-term, this feels useful.
Feedback offline from @rofrankel is to wait until apply is merged (issue tracked in #110)
blocked on #248 - once that is merged, this will also merge.