- Notifications
You must be signed in to change notification settings - Fork906
Closed
Description
Push is supposed to be a superset of create, so we're carrying significant code debt. The slight mismatches in their functionality are mostly mistakes, e.g.:
- create locally validates the template name whereas push does not
- create accepts TTLs whereas push does not
We should make sure to deprecate create for a period of time, perhaps with an annoyingtime.Sleep
. We should also changepush
to create a new template by default.
Bonus points for reducing duplication acrosstemplate edit
too. See#9319 for an example bug.