X-Git-Url: https://git.cworth.org/git?a=blobdiff_plain;f=turbot%2Fpuzzle.py;h=d24a7c9a4da955120f565a3ea43db4e8d66f61d5;hb=5f24de7e9cdb24f5bf751e3ce1d6d7333241747f;hp=caf6cede14e0a86ea6e4020ee4abbfd2a8bed745;hpb=cab4e4b9d7a0c37e988765c723a135739a3a3112;p=turbot diff --git a/turbot/puzzle.py b/turbot/puzzle.py index caf6ced..d24a7c9 100644 --- a/turbot/puzzle.py +++ b/turbot/puzzle.py @@ -249,8 +249,6 @@ def puzzle_channel_topic(puzzle): if puzzle['status'] == 'solved': topic += "SOLVED: `{}` ".format('`, `'.join(puzzle['solution'])) - topic += puzzle['name'] - links = [] url = puzzle.get('url', None) @@ -262,7 +260,7 @@ def puzzle_channel_topic(puzzle): links.append("<{}|Sheet>".format(sheet_url)) if len(links): - topic += "({})".format(', '.join(links)) + topic += "({}) ".format(', '.join(links)) tags = puzzle.get('tags', []) if tags: @@ -274,6 +272,37 @@ def puzzle_channel_topic(puzzle): return topic +def puzzle_channel_description(puzzle): + """Compute the channel description for a puzzle""" + + url = puzzle.get('url', None) + sheet_url = puzzle.get('sheet_url', None) + tags = puzzle.get('tags', []) + state = puzzle.get('state', None) + + description = ( + "Puzzle: \"{}\".\n".format(puzzle['name']) + ) + + links = '' + if url: + links += " <{}|Original puzzle> ".format(url) + + if sheet_url: + links += " <{}|Sheet>".format(sheet_url) + + if links: + description += "Links:{}\n".format(links) + + if tags: + description += "Tags: {}\n".format( + " ".join(["`{}`".format(t) for t in tags])) + + if state: + description += "State: {}\n".format(state) + + return description + def puzzle_channel_name(puzzle): """Compute the channel name for a puzzle""" @@ -327,6 +356,20 @@ def puzzle_update_channel_and_sheet(turb, puzzle, old_puzzle=None): turb.slack_client.conversations_setTopic(channel=channel_id, topic=channel_topic) + # Compute the channel description and set it if it has changed + channel_description = puzzle_channel_description(puzzle) + + old_channel_description = None + if old_puzzle: + old_channel_description = puzzle_channel_description(old_puzzle) + + if channel_description != old_channel_description: + # Slack also only allows 250 characters for a description + if len(channel_description) > 250: + channel_description = channel_description[:247] + "..." + turb.slack_client.conversations_setPurpose(channel=channel_id, + purpose=channel_description) + # Compute the sheet name and set it if it has changed sheet_name = puzzle_sheet_name(puzzle)