Simplified New publishing check + removed unnecessary Webhook check

- Replaced type checking with `TextChannel.is_news()` for simplification to check is possible to publish new
- Removed unnecessary `while` loop on `send_webhook` that check is webhook available. No need for this after starting ordering modification.

1 files changed, 2 insertions, 6 deletions

diff --git a/bot/cogs/news.py b/bot/cogs/news.py
index d7b2bcabb..66645bca7 100644
--- a/bot/cogs/news.py
+++ b/bot/cogs/news.py
@@ -116,7 +116,7 @@ class News(Cog):
             )
             payload["data"]["pep"].append(msg.id)
 
-            if msg.channel.type is discord.ChannelType.news:
+            if msg.channel.is_news():
                 log.trace("Publishing PEP annnouncement because it was in a news channel")
                 await msg.publish()
 
@@ -170,7 +170,7 @@ class News(Cog):
                 )
                 payload["data"][maillist].append(msg.id)
 
-                if msg.channel.type is discord.ChannelType.news:
+                if msg.channel.is_news():
                     log.trace("Publishing mailing list message because it was in a news channel")
                     await msg.publish()
 
@@ -223,10 +223,6 @@ class News(Cog):
             )
         embed.set_footer(text=footer, icon_url=AVATAR_URL)
 
-        # Wait until Webhook is available
-        while not self.webhook:
-            pass
-
         return await self.webhook.send(
             embed=embed,
             username=webhook_profile_name,