turbot
4 months agoAdd a stub for the turbot web view master
Carl Worth [Wed, 12 Jan 2022 06:45:07 +0000 (22:45 -0800)]
Add a stub for the turbot web view

Basically just to ensure we can distinguish a Slack request from any
other.

4 months agoTODO: Drop a bunch of items that have been completed
Carl Worth [Wed, 12 Jan 2022 05:55:12 +0000 (21:55 -0800)]
TODO: Drop a bunch of items that have been completed

It always feels great to check off items from a TODO list!

4 months agoAdd a state field to the "edit hunt" form
Carl Worth [Wed, 12 Jan 2022 05:37:54 +0000 (21:37 -0800)]
Add a state field to the "edit hunt" form

Also, put both the hunt url and the hunt state into the hunt channel's topic.

4 months agoAdd a list of users that don't want to be auto-invited to new channels
Carl Worth [Wed, 12 Jan 2022 04:27:43 +0000 (20:27 -0800)]
Add a list of users that don't want to be auto-invited to new channels

Starting the list of initially with Lavi's user ID.

4 months agoAdd code to auto-invite all users in a hunt to each new puzzle channel
Carl Worth [Wed, 12 Jan 2022 04:16:15 +0000 (20:16 -0800)]
Add code to auto-invite all users in a hunt to each new puzzle channel

We had originally written the code this way, long ago, but disabled it
(prior to MH 2021) when some people found it intrusive to be invited
to every channel, (rather than selecting the puzzles they were
interested in participating in). And after disabling it we also
deleted the code entirely in commit
ef8d8f359be149adbbdbeda88ec4b6bbbebf928e

Prior to MH 2022 we had talked about adding a user preference since
some people would like to be auto-invited. But then we got a lot less
code written than we'd wanted to before MH 2022.

So we decided to add the auto-join feature back, (so that people could
see channels that have huddle icons for easily determining puzzles
that have active activity).

4 months agoAbbreviate round name to only 7 characters in Slack channel name
Carl Worth [Wed, 12 Jan 2022 03:58:21 +0000 (19:58 -0800)]
Abbreviate round name to only 7 characters in Slack channel name

Some users were nervous about the round portion of the channel name
overwhelming the space available for channel names, making it
impossible to distinguish one puzzle channel from another in the same
round. Restricting the round prefix to only 7 letters should help
mitigate that.

4 months agoAnnounce to the hunt whenever a puzzle receives a new tag
Carl Worth [Sat, 8 Jan 2022 10:46:43 +0000 (02:46 -0800)]
Announce to the hunt whenever a puzzle receives a new tag

This will hopefully help people stay abreast of puzzles that they are
most interested in (by having a central place to see which tags are
being added to puzzles).

4 months agoAdd tags field to the new_puzzle and edit_puzzle forms
Carl Worth [Sat, 8 Jan 2022 10:12:44 +0000 (02:12 -0800)]
Add tags field to the new_puzzle and edit_puzzle forms

This completes the puzzle edit form so that all state that can be
manipulated with slash commands can also be directly edited with the
interactive dialog form.

4 months agoAdd a /delete command
Carl Worth [Sat, 8 Jan 2022 09:36:17 +0000 (01:36 -0800)]
Add a /delete command

Which performs automated archiving of all channels for an inactive hunt.

4 months agoDon't set a channel description longer than 250 character.
Carl Worth [Sat, 8 Jan 2022 09:20:02 +0000 (01:20 -0800)]
Don't set a channel description longer than 250 character.

Oops. The Slack channel description _is_ just as limited as the
channel topic, (only 250 characters). So we have to trim it just like
we trim the channel topic.

At least the description has the title first, so different information
will appear in the topic and description and hopefully everything
necessary at least appears in one place or the other.

4 months agoUse common code for setting the channel topic and description
Carl Worth [Sat, 8 Jan 2022 09:02:51 +0000 (01:02 -0800)]
Use common code for setting the channel topic and description

Previously, the initial setting of the topic and description was with
custom code, separate from the code that set the topic later due to
puzzle edits. Because of this code duplication, the formatting of
these two sets of code had drifted already. Making the initial setting
of topic use the existing puzzle_channel_topic() function eliminates
the code duplication.

Similarly, we give the channel description the same treatment that we
were already giving the channel topic. That is, we update it whenever
a puzzle edit would cause it to change. While we're doing that, and
since the puzzle description doesn't have the same short limit as the
channel topic, we expand it to be longer, (previously it was the same
text as the initial channel topic). So the new
puzzle_channel_description constructs a more verbose description which
now includes the puzzle title (which the channel topic no longer
includes after the previous commit), and also some helpful message
about links and commands.

4 months agoDon't put the puzzle name into the channel topic
Carl Worth [Sat, 8 Jan 2022 09:01:31 +0000 (01:01 -0800)]
Don't put the puzzle name into the channel topic

Since hunt writers do various things to torture us, (see Infinite
Corridor), and since the channel topic has a small limit, don't put
the puzzle title into the topic. Otherwise, a long puzzle title could
force the important links to not appear at all in the topic.

4 months agopuzzle_copy: Copy the solution list as well
Carl Worth [Sat, 8 Jan 2022 08:27:11 +0000 (00:27 -0800)]
puzzle_copy: Copy the solution list as well

Otherwise, the new puzzle will be pointing at the old puzzle's
solution list, (and the code that looks to see if the solution list
has changed won't notice any change).

This fixes things so that using "/solved" to add a second (or third,
...) to a puzzle will now update the channel's topic, which wasn't
happening before.

4 months agoFix a typo in the message announcing a puzzle solution
Carl Worth [Sat, 8 Jan 2022 08:26:34 +0000 (00:26 -0800)]
Fix a typo in the message announcing a puzzle solution

This typo was visible to users and annoying ungrammatical.

4 months agoFix a type in a comment
Carl Worth [Sat, 8 Jan 2022 08:26:10 +0000 (00:26 -0800)]
Fix a type in a comment

Just had the word "the" where I meant to have "then".

4 months agoUse empty list instead of empty tuple in a few places
Carl Worth [Sat, 8 Jan 2022 08:25:01 +0000 (00:25 -0800)]
Use empty list instead of empty tuple in a few places

I don't think the type actually mattered anywhere here, but it might have.
(Leave it to python and its annoying type unsafety to be unhelpful here.)

4 months agoConvert the solution data structure back to a list, (but still avoid dupes.)
Carl Worth [Sat, 8 Jan 2022 02:55:32 +0000 (18:55 -0800)]
Convert the solution data structure back to a list, (but still avoid dupes.)

The put_item call was not happy receiving a set object. So, instead of using
one, we use a lits and just take care to not add any duplicates to the list.

4 months agoUse a set instead of a list for a puzzle's solution
Carl Worth [Sat, 8 Jan 2022 02:31:33 +0000 (18:31 -0800)]
Use a set instead of a list for a puzzle's solution

This allows for cleaner handling when two solvers submit the same
solution back-to-back. Previously the identical solution would appear
multiple times in the list of solutions which was obviously not
helpful.

Now, each solution will appear no more than once.

4 months agoFix puzzle edit form in the case of no existing rounds in the hunt yet
Carl Worth [Sat, 8 Jan 2022 01:26:59 +0000 (17:26 -0800)]
Fix puzzle edit form in the case of no existing rounds in the hunt yet

This was a corner case affecting only the first few puzzles or so, (before
any puzzles were associated with a round).

4 months agoRemove the util code
Carl Worth [Sat, 8 Jan 2022 00:36:14 +0000 (16:36 -0800)]
Remove the util code

This was getting in the way of deployment, (since it's not flake8-clean).

Instead of fixing it, just nuke it. We weren't certain we'd ever use it
anyway, (and it's now at least in git revision history if needed).

4 months agoMove all Turbot-created sheets down one level (in google drive)
Carl Worth [Sat, 8 Jan 2022 00:34:32 +0000 (16:34 -0800)]
Move all Turbot-created sheets down one level (in google drive)

The new folder is named "turbot"

4 months agoAdd some utility code
Carl Worth [Fri, 7 Jan 2022 03:28:42 +0000 (19:28 -0800)]
Add some utility code

None of this is really "part of" turbot, but is instead some
standalone code I've written from time to time either while testing
functionality or when live-tweaking something during hunt as follows:

sheets-test.py: Ensuring Google sheets API works as expected

blocks-test.py: Debugging some Slack blocks formatting

hunt-channel-topic-tweak.py: During MH 2021 it was useful (for
a reason I don't recall at the moment) to be able to manually
tweak the topic of some channels from a script and I whipped
these up at the moment.

It's not clear if any of this will be useful going forward, but just
in case I'm holding onto it for now. We may just want to delete this
code if we find we don't end up using it.

4 months agoIgnore the .gsheets.env file if it exists
Carl Worth [Fri, 7 Jan 2022 03:27:14 +0000 (19:27 -0800)]
Ignore the .gsheets.env file if it exists

I've had this around for some utility code and don't want "git status"
saying anything about it.

4 months agoEdit the help messages for "new" and "edit"
Carl Worth [Fri, 7 Jan 2022 03:24:18 +0000 (19:24 -0800)]
Edit the help messages for "new" and "edit"

The old help messages were confusing. It wasn't always clear that
"[puzzle]" meant an optional argument of literally the word "puzzle"
rather than the name of a particular puzzle to edit.

Instead, we encourage people to use these commands without arguments
(whenever possible, such as "/edit" for hunt or puzzle and "/new" for
puzzle) and to use clear literal arguments when necessary (such as
"/new hunt" from any channel or "/edit hunt" from within a puzzle
channel).

4 months agoAdd a README file for turbot
Carl Worth [Fri, 7 Jan 2022 00:55:47 +0000 (16:55 -0800)]
Add a README file for turbot

If nothing else, this will be helpful to remind me what the email
addresses are that I've been using for AWS and Google Drive accounts,
and also what the URL is for editing the Turbot app.

6 months agoRemove some recently-added debugging print statements
Carl Worth [Mon, 15 Nov 2021 23:54:31 +0000 (15:54 -0800)]
Remove some recently-added debugging print statements

After determining the actual problem.

6 months agoFix "New Hunt" button to actually call new_hunt_button function
Carl Worth [Mon, 15 Nov 2021 23:52:22 +0000 (15:52 -0800)]
Fix "New Hunt" button to actually call new_hunt_button function

Was previously calling new_hunt function directly which cannot work.

6 months agoAdd some debugging of the turbot home app
Carl Worth [Mon, 15 Nov 2021 23:27:32 +0000 (15:27 -0800)]
Add some debugging of the turbot home app

I'm getting some "channel_not_found" errors that I don't understand,
because I don't think I have any active hunts currently.

6 months agoAdd a couple of more TODO items
Carl Worth [Mon, 15 Nov 2021 22:20:36 +0000 (14:20 -0800)]
Add a couple of more TODO items

These were left over from the Hunt in January 2021.

16 months agoFix missing case of nested array
Carl Worth [Sat, 16 Jan 2021 16:11:23 +0000 (08:11 -0800)]
Fix missing case of nested array

Hoping we're there now with this fix?

16 months agoRestrict list of blocks to 100 items
Carl Worth [Sat, 16 Jan 2021 16:03:10 +0000 (08:03 -0800)]
Restrict list of blocks to 100 items

Since I've now seen Slack complaining if lists are greater than 100 items.

16 months agoLive fix for apparently overflowing the Slack message limit with hunt details
Carl Worth [Sat, 16 Jan 2021 15:47:45 +0000 (07:47 -0800)]
Live fix for apparently overflowing the Slack message limit with hunt details

Condense the Turbot Home view to only link to hunts, not give all the
details.

Change hunt_blocks() function to return an array of array of blocks,
(broken up at the level of a round). Callers now generate one post for
each element in the outer array. Hopefully this gives us a little
breathing room for now.

16 months agoAdd some recent TODO items
Carl Worth [Sat, 16 Jan 2021 15:44:27 +0000 (07:44 -0800)]
Add some recent TODO items

These obviously aren't going to happen or this year, but we can
consider them for future years.

16 months agoWhen creating a new puzzle, set default rounds based on current puzzle
Carl Worth [Wed, 13 Jan 2021 04:15:46 +0000 (20:15 -0800)]
When creating a new puzzle, set default rounds based on current puzzle

This happens when running the puzzle creation command, (/new,
/puzzle new, or /new puzzle), from an existing channel.

This should hopefully help people avoid forgetting to set the rounds
for a new puzzle.

16 months agoSwitch from using append() to update() for inserting into a spreadsheet
Carl Worth [Wed, 13 Jan 2021 04:03:36 +0000 (20:03 -0800)]
Switch from using append() to update() for inserting into a spreadsheet

The append() method was the entirely wrong thing: It puts data in as
new rows below existing data. The update() method does what we want,
(sets data at a specific range).

16 months agoUse a double dash separator for the m in the channel name of metapuzzles
Carl Worth [Tue, 12 Jan 2021 02:06:10 +0000 (18:06 -0800)]
Use a double dash separator for the m in the channel name of metapuzzles

The double dash is what will make the meta puzzles sort before all
non-metapuzzled.

16 months agoUse consistent channel naming at puzzle creation time
Carl Worth [Tue, 12 Jan 2021 01:56:47 +0000 (17:56 -0800)]
Use consistent channel naming at puzzle creation time

We've had a bug for some time where the puzzle editing code would
construct a name for the puzzle channel based on hunt, and round, and
puzzle type, and puzzle_id but the puzzle creation code was only using
hunt and puzzle_id.

In this commit, we fix the puzzle creation code to use the same
puzzle_channel_name function so that channel names are created
consistently.

This commit also adds a check to reject a puzzle if it has the same
puzzle_id as an existing puzzle.

16 months agoAdd find_puzzle_for_puzzle_id
Carl Worth [Tue, 12 Jan 2021 01:53:36 +0000 (17:53 -0800)]
Add find_puzzle_for_puzzle_id

This exists so that the puzzle-creation code can ensure that it's not
creating a puzzle with the same puzzle_id as an existing
puzzle. Previously, Turbot was just relying on Slack to guarantee
uniqueness, (which worked when channel names were just made from
hunt_id and puzzle_id).

But more recently, the channel names have round names in them as
well. This set up a possible collision where two puzzle with the same
puzzle_id could be created but with different channel names, (if the
two puzzles were placed in different rounds). To be able to fix this
bug we need this query to be able to find a puzzle by means of using
the puzzle_id.

This does require a new index on the database, so that's here too,
(and we're nuking our database to be able to roll this out).

16 months agoFix comment string for find_puzzle_for_sort_key
Carl Worth [Tue, 12 Jan 2021 01:52:40 +0000 (17:52 -0800)]
Fix comment string for find_puzzle_for_sort_key

The documentation here was stale and still referring to puzzle_id even
though this function is actually using sort_key, (which is similar but
is also slightly different).

16 months agoTighten up the welcome messages for a new hunt and a new puzzle
Carl Worth [Tue, 12 Jan 2021 00:50:29 +0000 (16:50 -0800)]
Tighten up the welcome messages for a new hunt and a new puzzle

And point to `/help` in both cases for more details.

16 months agoExpand list of pop-culture items to include video games
Carl Worth [Mon, 11 Jan 2021 22:25:06 +0000 (14:25 -0800)]
Expand list of pop-culture items to include video games

Perhaps bringing this list more up-to-date, (and also quite compatible
with the proclivities of ✈✈✈ Galactic Trendsetters ✈✈✈ who are writing
this year).

16 months agoDrop names of specific people from the have-you-tried list
Carl Worth [Mon, 11 Jan 2021 22:24:27 +0000 (14:24 -0800)]
Drop names of specific people from the have-you-tried list

Replacing this list with "relevant team experts".

16 months agoFix `/help me` to not give a dispatch_failed error in a private channel
Carl Worth [Mon, 11 Jan 2021 21:56:20 +0000 (13:56 -0800)]
Fix `/help me` to not give a dispatch_failed error in a private channel

Since Turbot can't send a message to a private channel, instead we
just reply as an ephemeral message to the user.

16 months agoMake "/help me" self-documenting
Carl Worth [Mon, 11 Jan 2021 21:09:52 +0000 (13:09 -0800)]
Make "/help me" self-documenting

So that when people see others doing this, they will know how to do it too.

16 months agoImplement a new "/help me" command that suggests something to try
Carl Worth [Mon, 11 Jan 2021 20:53:34 +0000 (12:53 -0800)]
Implement a new "/help me" command that suggests something to try

Thanks to Acme for the original "Have you tried" list as obtained from:

    http://web.mit.edu/puzzle/www/resources.html

as:

    http://web.mit.edu/puzzle/www/resources/haveyoutried.pdf

16 months agoAdd an initial implementation of /help
Carl Worth [Mon, 11 Jan 2021 20:21:29 +0000 (12:21 -0800)]
Add an initial implementation of /help

So that Turbot can be self-documenting.

16 months agoUpdate hunt welcome message to refer to `/puzzle new`
Carl Worth [Mon, 11 Jan 2021 18:09:38 +0000 (10:09 -0800)]
Update hunt welcome message to refer to `/puzzle new`

Previously it was pointing the users at `/puzzle` which is no longer
the correct command to use.

16 months agoFix call to find_hunt_for_hunt_id
Carl Worth [Mon, 11 Jan 2021 17:58:12 +0000 (09:58 -0800)]
Fix call to find_hunt_for_hunt_id

It takes two parameters, not one, (was missing "turb" argument).

This should fix the edit_hunt button (the pencil) which was erroring
out while this bug was present.

16 months agoDrop use of response_url in button handlers
Carl Worth [Mon, 11 Jan 2021 17:54:57 +0000 (09:54 -0800)]
Drop use of response_url in button handlers

Some buttons appear in a context, (like the app's "Home" view), where
there is no response_url, (we can't generate a message there). So we
can't rely on this. And we don't need to anyway, we can use the string
in the reteurn value to get an error message out to the user.

This commit should fix a bunch of broken puzzle edits button in the
Turbot "home" view.

16 months agoUpdate TODO now that "/hunt edit" has been added
Carl Worth [Mon, 11 Jan 2021 17:01:08 +0000 (09:01 -0800)]
Update TODO now that "/hunt edit" has been added

This closes out the last item we had as a required change prior to
Mystery Hunt 2021.

16 months agoMake /edit command smart to auto-edit either puzzle or hunt
Carl Worth [Mon, 11 Jan 2021 16:25:20 +0000 (08:25 -0800)]
Make /edit command smart to auto-edit either puzzle or hunt

Depending on which kind of channel it is run in. It's still possible to
explicitly run `/edit puzzle` or `/edit hunt` instead.

16 months agoAdd a pencil-shaped "edit hunt" button when displaying a hunt
Carl Worth [Mon, 11 Jan 2021 16:07:03 +0000 (08:07 -0800)]
Add a pencil-shaped "edit hunt" button when displaying a hunt

So that editing a hunt can be just as convenient as editing a puzzle.

16 months agoWire up "/edit hunt" to be the same as "/hunt edit"
Carl Worth [Mon, 11 Jan 2021 16:03:30 +0000 (08:03 -0800)]
Wire up "/edit hunt" to be the same as "/hunt edit"

Just as we did for "/edit" and "/edit puzzle" before.

16 months agoAdd a "/hunt edit" command
Carl Worth [Mon, 11 Jan 2021 16:00:40 +0000 (08:00 -0800)]
Add a "/hunt edit" command

We expect this to be useful for a situation like Mystery Hunt where we
create the hunt and its channel in advance, invite all the users to
it, and then only later do we learn what the URL of the hunt actually
is.

16 months agoAdd "/new hunt" as a new sub-command of "/new"
Carl Worth [Sun, 10 Jan 2021 23:41:57 +0000 (15:41 -0800)]
Add "/new hunt" as a new sub-command of "/new"

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.

16 months agoUpdate TOOD with two recently-completed items
Carl Worth [Sun, 10 Jan 2021 21:33:17 +0000 (13:33 -0800)]
Update TOOD with two recently-completed items

There is a new "/hunt new" command now and puzzle sheets are now created
in a folder named after the hunt.

16 months agoCreate puzzle sheets in the hunt folder
Carl Worth [Sun, 10 Jan 2021 21:24:25 +0000 (13:24 -0800)]
Create puzzle sheets in the hunt folder

This will be much nicer to keep our content organized within Google
drive. There will be one folder for each hunt rather than all the
various hunts' puzzles all mixed together at the root level.

16 months agoUse the Google drive API to create a hunt's sheet within a folder
Carl Worth [Sun, 10 Jan 2021 17:52:27 +0000 (09:52 -0800)]
Use the Google drive API to create a hunt's sheet within a folder

When a hunt is created, we create a folder in Google drive named
after the hunt_id and the create the hunt's sheet in that folder.

In future commits, we'll change the puzzle sheets to also be created
in this hunt's folder. This way, with all resources in one folder,
that folder can be shared out to members of the hunt, (this could
be used as an alternate access to hunt resources if Slack is not
available for any reason).

16 months agoRename identifier from item to hunt
Carl Worth [Sun, 10 Jan 2021 17:51:15 +0000 (09:51 -0800)]
Rename identifier from item to hunt

The dynamodb API uses the generic term "item", but our code is more
clear with the more distinctive term "hunt" for the identifier here.

This commit has no functional change.

16 months agoImplement "/hunt new" as a way to make a new hunt
Carl Worth [Sun, 10 Jan 2021 16:36:05 +0000 (08:36 -0800)]
Implement "/hunt new" as a way to make a new hunt

This is more convenient than seeking out the Turbot app's Home tab.

16 months agoRemove a line of dead code
Carl Worth [Sun, 10 Jan 2021 16:35:05 +0000 (08:35 -0800)]
Remove a line of dead code

A second return statement after an unconditional return statement
obviously cannot do anything.

16 months agoFix to work with old puzzles that don't have a type attribute
Carl Worth [Sat, 9 Jan 2021 22:39:39 +0000 (14:39 -0800)]
Fix to work with old puzzles that don't have a type attribute

Without this, commands like /state and /puzzle on old puzzle just die.

16 months agoAllow tag values to have numbers
Carl Worth [Sat, 9 Jan 2021 22:37:10 +0000 (14:37 -0800)]
Allow tag values to have numbers

There was unanimous consent that this could be useful, and it's an
easy change.

16 months agoAdd 4 new TODO items
Carl Worth [Sat, 9 Jan 2021 22:36:18 +0000 (14:36 -0800)]
Add 4 new TODO items

Just the last few fixes we've identified we want to have in place
before the hunt next week.

16 months agoPut the puzzle's name into the first tab of the sheet
Carl Worth [Sat, 9 Jan 2021 16:06:31 +0000 (08:06 -0800)]
Put the puzzle's name into the first tab of the sheet

Just to make it that much easier to tell apart multiple sheet windows
that a user might have to juggle.

16 months agoUpdate the TODO list for a couple of recently-completed items
Carl Worth [Sat, 9 Jan 2021 16:03:15 +0000 (08:03 -0800)]
Update the TODO list for a couple of recently-completed items

We got rid of that "Copy of" text in the sheet tabs, and we also put
the round name into the Slack channel.

16 months agoAdd both round and "-m" (for META), if appropriate, into channel name
Carl Worth [Sat, 9 Jan 2021 15:52:41 +0000 (07:52 -0800)]
Add both round and "-m" (for META), if appropriate, into channel name

For better grouping of related puzzles in the channel list

16 months agoUse best-practice <@user_id> to mention a user in Slack
Carl Worth [Sat, 9 Jan 2021 15:45:37 +0000 (07:45 -0800)]
Use best-practice <@user_id> to mention a user in Slack

This way Slack expands this as a link to the user's full name.

16 months agoCollapse link text inserted into sheets
Carl Worth [Sat, 9 Jan 2021 15:31:13 +0000 (07:31 -0800)]
Collapse link text inserted into sheets

Since we have functional links here we don't need to include the full
URL as part of the link text.

16 months agoRewrite all of sheets.py to look a little more like turbot code
Carl Worth [Sat, 9 Jan 2021 15:19:21 +0000 (07:19 -0800)]
Rewrite all of sheets.py to look a little more like turbot code

The only actual intended behavioral change here is that the puzzle
sheet's now nuke the blank first sheet, (and instead have only sheets
copied over from the template).

16 months agoRename each sheet we copy from the template
Carl Worth [Sat, 9 Jan 2021 14:29:42 +0000 (06:29 -0800)]
Rename each sheet we copy from the template

To get rid of the silly "Copy of" text that we don't need.

16 months agoFix sheet creation to copy all sheets from our template
Carl Worth [Sat, 9 Jan 2021 13:47:33 +0000 (05:47 -0800)]
Fix sheet creation to copy all sheets from our template

Nathan put a lot of word into this template sheet, so it was a shame
the sheets weren't all coming over. It turned out the only reason that
was the case is that the code has a hard-coded list of names of sheets
to copy, (and that, only because I haven't yet figured out the API for
enumerating sheets).

For now, just update the hardcoded list to match what's in the current
template.

16 months agoUpdate TODO for recent addition of /tag command
Carl Worth [Sat, 9 Jan 2021 13:42:59 +0000 (05:42 -0800)]
Update TODO for recent addition of /tag command

This _almost_ completed the feature as intended. In fact, it went
further in one way in that it's already storing the tags in a field
that is separate from the state string. But it's falling short in that
the edit interface is not currently showing the tags at all.

16 months agoAdd hint to expand search terms if implicitly filtered
Carl Worth [Sat, 9 Jan 2021 13:39:30 +0000 (05:39 -0800)]
Add hint to expand search terms if implicitly filtered

I keep accidentally missing some search results when testing because
I'm searching for a tag that only exists on a solved puzzle so something
like:

/hunt cryptic

is not finding the solved puzzle with the CRYTPIC tag due to the
implicit filtering to only unsolved puzzles.

I still want that implicit filtering, but maybe this nudge will be
helpful in cases like this.

16 months agoExtend puzzle search to include type and tags
Carl Worth [Sat, 9 Jan 2021 13:21:43 +0000 (05:21 -0800)]
Extend puzzle search to include type and tags

The matching on puzzle type means you can now do:

/hunt meta

or:

/hunt plain

to see all meta or non-meta puzzles.

And the matching on tags should be obvious.

16 months agoUse a deep copy when comparing a puzzle to detect state modifications
Carl Worth [Sat, 9 Jan 2021 13:16:20 +0000 (05:16 -0800)]
Use a deep copy when comparing a puzzle to detect state modifications

Since the recent tags is an array, we need a deep copy of the puzzle dict
in order to be able to reliably compare the puzzle prior to the copy with
the one after the copy where a tag has been added or removed.

With this fix, adding or removing a tag now reliably updates the channel
description.

16 months agoAdd a /tag command to add or remove tags from puzzle
Carl Worth [Sat, 9 Jan 2021 13:04:28 +0000 (05:04 -0800)]
Add a /tag command to add or remove tags from puzzle

And put the tag values everywhere we have the state string already.

16 months agoAdd two new shortcut commands:
Carl Worth [Sat, 9 Jan 2021 12:27:35 +0000 (04:27 -0800)]
Add two new shortcut commands:

/new Which is the same as: /puzzle new
        /edit Which is the same as: /puzzle edit

I don't know if this is cleaner or messier, but I can at least put it
out there as an option.

16 months agoFor /puzzle on meta puzzles print all titles/answers of feeder puzzles
Carl Worth [Sat, 9 Jan 2021 11:59:02 +0000 (03:59 -0800)]
For /puzzle on meta puzzles print all titles/answers of feeder puzzles

Where feeder puzzles are specifically defined as puzzle belonging to the
same round.

16 months agoPut solution after puzzle name in /hunt reports
Carl Worth [Sat, 9 Jan 2021 11:27:27 +0000 (03:27 -0800)]
Put solution after puzzle name in /hunt reports

The concept here is to keep the puzzle names aligned to make it easier
to find a puzzle of interest in the sorted list.

16 months agoAdd a divider between rounds in the /hunt output
Carl Worth [Sat, 9 Jan 2021 11:22:25 +0000 (03:22 -0800)]
Add a divider between rounds in the /hunt output

Just to make it a little easier to see the grouping of rounds.

16 months agoTODO: Update for two recently-completed meta items
Carl Worth [Sat, 9 Jan 2021 11:20:34 +0000 (03:20 -0800)]
TODO: Update for two recently-completed meta items

The punch list is getting short now...

16 months agoSort all meta puzzles before all non-meta puzzle in puzzle lists
Carl Worth [Sat, 9 Jan 2021 10:57:07 +0000 (02:57 -0800)]
Sort all meta puzzles before all non-meta puzzle in puzzle lists

This is in the output of /hunt, /round, and the Turbot home view.

16 months agonew_puzzle_submission: Create a puzzle dict before shoving it into the DB
Carl Worth [Sat, 9 Jan 2021 10:47:57 +0000 (02:47 -0800)]
new_puzzle_submission: Create a puzzle dict before shoving it into the DB

This will be handy for calling some puzzle.py functions that expect a dict.

16 months agoAdd a "meta" checkbox when creating/editing a puzzle
Carl Worth [Sat, 9 Jan 2021 10:36:20 +0000 (02:36 -0800)]
Add a "meta" checkbox when creating/editing a puzzle

And display all meta puzzles with a bold "META" prefix.

16 months agoFix a typo in a comment
Carl Worth [Sat, 9 Jan 2021 10:35:23 +0000 (02:35 -0800)]
Fix a typo in a comment

This comment was supposed to refer to puzzle_id, not hunt_id, which
made it confusing for me until I fixed it as in this commit.

16 months agoImplement a /round command
Carl Worth [Sat, 9 Jan 2021 09:19:57 +0000 (01:19 -0800)]
Implement a /round command

This is much like the /hunt command with searching, etc. but with two
differences:

  1. It is limited to display puzzles in the same round(s) as the
     current puzzle

  2. It defaults to display all puzzles rather than unsolved puzzles
     like /hunt does

16 months agoImplement a `/puzzle edit` command
Carl Worth [Sat, 9 Jan 2021 08:39:57 +0000 (00:39 -0800)]
Implement a `/puzzle edit` command

As a simple shortcut rather than making the user type `/puzzle` and
then click the pencil icon.

16 months agoReport change in a puzzle's solved status to the main hunt channel
Carl Worth [Sat, 9 Jan 2021 08:32:18 +0000 (00:32 -0800)]
Report change in a puzzle's solved status to the main hunt channel

We were already doing this from the /solved command but now we do it
for puzzle edits as well.

16 months agoPut a message into the channel when a user edits a puzzle
Carl Worth [Sat, 9 Jan 2021 08:26:24 +0000 (00:26 -0800)]
Put a message into the channel when a user edits a puzzle

So that other members of the puzzle are informed that something has changed.

16 months agoAdd display of round(s) in /puzzle output
Carl Worth [Sat, 9 Jan 2021 08:00:22 +0000 (00:00 -0800)]
Add display of round(s) in /puzzle output

Making the output more complete (while still remaining quite compact).

16 months agoAdd new /puzzle command which simply prints the current puzzle
Carl Worth [Sat, 9 Jan 2021 07:51:32 +0000 (23:51 -0800)]
Add new /puzzle command which simply prints the current puzzle

This is the same format one could get from the /hunt command, but
limited to the current puzzle. This is a very convenient way to see
all the puzzle fields and to get access to the pencil button to edit
the current puzzle.

16 months agoRename puzzle creation command from "/puzzle" to "/puzzle new"
Carl Worth [Sat, 9 Jan 2021 07:36:32 +0000 (23:36 -0800)]
Rename puzzle creation command from "/puzzle" to "/puzzle new"

This is to create space for other commands later, (such as "/puzzle edit"
and a new status-reporting "/puzzle" command).

16 months agoDon't re-set channel and sheet name or channel topic to the same value as before
Carl Worth [Sat, 9 Jan 2021 07:04:55 +0000 (23:04 -0800)]
Don't re-set channel and sheet name or channel topic to the same value as before

The puzzle_update_channel_and_sheet function is modified here to
accept an old_puzzle argument in addition to the puzzle argument. By
using this it can decide whether there's any change in the sheet or
channel data before setting it.

This optimization avoids Slack reporting things like:

    Turbot renamed the channel from "hunt-puzzle" to "hunt-puzzle"

Which is obviously just noise.

16 months agoUnify code to rename channel, set channel description, and rename sheet
Carl Worth [Sat, 9 Jan 2021 06:38:26 +0000 (22:38 -0800)]
Unify code to rename channel, set channel description, and rename sheet

With this code, all of those changes are made consistently regardless of
whether the change happens from /state, /solved, or a puzzle edit (which
could set status to either "solved" or "unsolved").

16 months agoReject puzzle edit if "solved" and "solution" are inconsistent
Carl Worth [Sat, 9 Jan 2021 06:19:29 +0000 (22:19 -0800)]
Reject puzzle edit if "solved" and "solution" are inconsistent

There's an argument here that we really shouldn't be giving the user
two different fields if we're forcing them to be edited together, but
that's what we have right now.

16 months agoUpdate TODO file now that puzzle edit dialog has been edited
Carl Worth [Sat, 9 Jan 2021 06:18:26 +0000 (22:18 -0800)]
Update TODO file now that puzzle edit dialog has been edited

And just as the TODO file predicted, right now it's only possible to
get at this functionality through the pencil icon.

16 months agoImplement update of puzzle when edit_puzzle dialog is submitted
Carl Worth [Sat, 9 Jan 2021 02:44:29 +0000 (18:44 -0800)]
Implement update of puzzle when edit_puzzle dialog is submitted

Processing all user input and updating the database with the changes.

This isn't _quite_ updating everything in the channel and the sheet
that should be changed. To do that we simply need to refactor the code
that's currently doing that in various places to do it all in one
place.

16 months agoImplement a dialog box to edit a puzzle
Carl Worth [Sat, 9 Jan 2021 01:23:32 +0000 (17:23 -0800)]
Implement a dialog box to edit a puzzle

This is connected to the "pencil" buttons that were recently added to
the /hunt output as well as the Turbot home view.

So far, this brings up an editable dialog with the puzzle state but
doesn't actually do anything with any of the edits made when the user
saves the dialog.

16 months agoModify multi_select_block to use accessory_block, section_block, and text_block
Carl Worth [Fri, 8 Jan 2021 20:37:40 +0000 (12:37 -0800)]
Modify multi_select_block to use accessory_block, section_block, and text_block

Where this content was previously open-coded and redundant.

The commit removes more lines of code than it adds.