summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
540f66a)
This means we now have both "/hunt new" as well as "/new hunt" that
both do the same things. I'm not sure we'll want to keep both around
long-term, but they are both here now so that we can experiment and
see which feels more natural.
def new(turb, body, args):
"""Implementation of the `/new` command
def new(turb, body, args):
"""Implementation of the `/new` command
- To create a new puzzle.
-
- This is simply a shortcut for `/puzzle new`.
+ This can be used to create a new hunt ("/new hunt") or a new
+ puzzle ("/new puzzle" or simply "/new"). So puzzle creation is the
+ default behavior (as it is much more common).
+
+ This operations are identical to the existing "/hunt new" and
+ "/puzzle new". I don't know that that redundancy is actually
+ helpful in the interface. But at least having both allows us to
+ experiment and decide which is more natural and should be kept
+ around long-term.
+ if args == 'hunt':
+ return new_hunt_command(turb, body)
+
return new_puzzle(turb, body)
commands["/new"] = new
return new_puzzle(turb, body)
commands["/new"] = new