diff options
| author | 2019-01-07 20:12:16 -0500 | |
|---|---|---|
| committer | 2019-01-07 20:12:16 -0500 | |
| commit | 5e794a0a7ab732ff2ac2ba37473ecd87413603d4 (patch) | |
| tree | d3eda0698b5beacebbce1b29c2a90f52d03290d3 | |
| parent | Merge branch 'master' of https://github.com/python-discord/bot (diff) | |
Update method for mentioning users and channels to be more idiomatic
| -rw-r--r-- | bot/cogs/free.py | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/bot/cogs/free.py b/bot/cogs/free.py index 4566ade5f..71ee18032 100644 --- a/bot/cogs/free.py +++ b/bot/cogs/free.py @@ -51,14 +51,14 @@ class Free: inactive = (datetime.utcnow() - msg.created_at).seconds if inactive > self.TIME_INACTIVE: - free_channels.append((inactive, channel.id)) + free_channels.append((inactive, channel)) embed = Embed() embed.colour = Colour.gold() embed.title = "**Looking for a free help channel?**" if user is not None: - embed.description = f"**Hey <@{user.id}>!**\n\n" + embed.description = f"**Hey {user.mention}!**\n\n" else: embed.description = "" @@ -67,14 +67,13 @@ class Free: 's' if len(free_channels) > 1 else '', '' if len(free_channels) > 1 else 's') - for i, channel in enumerate(sorted(free_channels, reverse=True), 1): - inactive, ID = channel + for i, (inactive, channel) in enumerate(sorted(free_channels, reverse=True), 1): minutes, seconds = divmod(inactive, 60) if minutes > 60: hours, minutes = divmod(minutes, 60) - embed.description += f"{i}. <#{ID}> inactive for {hours}h{minutes}m{seconds}s\n\n" + embed.description += f"{i}. {channel.mention} inactive for {hours}h{minutes}m{seconds}s\n\n" else: - embed.description += f"{i}. <#{ID}> inactive for {minutes}m{seconds}s\n\n" + embed.description += f"{i}. {channel.mention} inactive for {minutes}m{seconds}s\n\n" embed.description += ("**\nThese channels aren't guaranteed to be free, " "so use your best judgement and check for yourself.") |