Merge pull request #145 from Pycord-Development/slash

merge slash into restructure

Author: BobDotCom

.github/CONTRIBUTING.md

@@ -34,6 +34,10 @@ If the bug report is missing this information then it'll take us longer to fix t
 
 Submitting a pull request is fairly simple, just make sure it focuses on a single aspect and doesn't manage to have scope creep and it's probably good to go. It would be incredibly lovely if the style is consistent to that found in the project. This project follows PEP-8 guidelines (mostly) with a column limit of 125.
 
+### Licensing
+
+By submitting a pull request, you agree that; 1) You hold the copyright on all submitted code inside said pull request; 2) You agree to transfer all rights to the owner of this repository, and; 3) If you are found to be in fault with any of the above, we shall not be held responsible in any way after the pull request has been merged.
+
 ### Git Commit Guidelines
 
 - Use present tense (e.g. "Add feature" not "Added feature")

README.rst

@@ -72,22 +72,20 @@ Quick Example
 
     import discord
 
-    class MyClient(discord.Client):
-        async def on_ready(self):
-            print("Logged on as", self.user)
-
-        async def on_message(self, message):
-            # don't respond to ourselves
-            if message.author == self.user:
-                return
-
-            if message.content == "ping":
-                await message.channel.send("pong")
-
-    client = MyClient()
-    client.run("token")
+    bot = discord.Bot()
+    
+    @bot.slash_command()
+    async def hello(ctx, name: str = None):
+        name = name or ctx.author.name
+        await ctx.send(f"Hello {name}!")
+        
+    @bot.user_command(name="Say Hello")
+    async def hi(ctx, user):
+        await ctx.send(f"{ctx.author.mention} says hello to {user.name}!")
+        
+    bot.run("token")
 
-Bot Example
+Normal Commands Example
 ~~~~~~~~~~~~~
 
 .. code:: py
@@ -110,4 +108,5 @@ Links
 
 - `Documentation <https://pycord.readthedocs.io/en/latest/index.html>`_
 - `Official Discord Server <https://discord.gg/dK2qkEJ37N>`_
+- `Discord Developers <https://discord.gg/discord-developers>`_
 - `Discord API <https://discord.gg/discord-api>`_

discord/__init__.py

@@ -62,6 +62,8 @@
 from .bot import *
 from .app import *
 from .cog import Cog
+from .welcome_screen import *
+
 
 class VersionInfo(NamedTuple):
     major: int

discord/abc.py

@@ -62,6 +62,7 @@
     'GuildChannel',
     'Messageable',
     'Connectable',
+    'Mentionable'
 )
 
 T = TypeVar('T', bound=VoiceProtocol)
@@ -1141,6 +1142,7 @@ class Messageable:
     - :class:`~discord.Member`
     - :class:`~discord.ext.commands.Context`
     - :class:`~discord.Thread`
+    - :class:`~discord.ApplicationContext`
     """
 
     __slots__ = ()
@@ -1690,4 +1692,4 @@ async def connect(
 
 class Mentionable:
     # TODO: documentation, methods if needed
-    pass
+    pass

discord/app/commands.py

@@ -405,13 +405,13 @@ async def _invoke(self, ctx: ApplicationContext) -> None:
                 <= SlashCommandOptionType.role.value
             ):
                 name = "member" if op.input_type.name == "user" else op.input_type.name
-                arg = await get_or_fetch(ctx.guild, name, int(arg))
+                arg = await get_or_fetch(ctx.guild, name, int(arg), default=int(arg))
 
             elif op.input_type == SlashCommandOptionType.mentionable:
-                try:
-                    arg = await get_or_fetch(ctx.guild, "member", int(arg))
-                except NotFound:
-                    arg = await get_or_fetch(ctx.guild, "role", int(arg))
+                arg_id = int(arg)
+                arg = await get_or_fetch(ctx.guild, "member", arg_id)
+                if arg is None:
+                    arg = ctx.guild.get_role(arg_id) or arg_id
 
             kwargs[op.name] = arg
 
@@ -466,7 +466,7 @@ def _update_copy(self, kwargs: Dict[str, Any]):
 
 class Option:
     def __init__(
-        self, input_type: Any, /, description = None,**kwargs
+        self, input_type: Any, /, description: str = None, **kwargs
     ) -> None:
         self.name: Optional[str] = kwargs.pop("name", None)
         self.description = description or "No description provided"
@@ -501,9 +501,11 @@ def __init__(self, name: str, value: Optional[Union[str, int, float]] = None):
     def to_dict(self) -> Dict[str, Union[str, int, float]]:
         return {"name": self.name, "value": self.value}
 
-def option(name, type, **kwargs):
+def option(name, type=None, **kwargs):
     """A decorator that can be used instead of typehinting Option"""
     def decor(func):
+        nonlocal type
+        type = type or func.__annotations__.get(name, str)
         func.__annotations__[name] = Option(type, **kwargs)
         return func
     return decor

discord/app/context.py

@@ -24,19 +24,21 @@
 
 from typing import TYPE_CHECKING, Optional, Union
 
+import discord.abc
+
 if TYPE_CHECKING:
     import discord
+    from discord.state import ConnectionState
 
 from ..guild import Guild
 from ..interactions import Interaction, InteractionResponse
 from ..member import Member
 from ..message import Message
 from ..user import User
 from ..utils import cached_property
-from ..context_managers import Typing
 
 
-class ApplicationContext:
+class ApplicationContext(discord.abc.Messageable):
     """Represents a Discord interaction context.
 
     This class is not created manually and is instead passed to application
@@ -58,6 +60,10 @@ def __init__(self, bot: "discord.Bot", interaction: Interaction):
         self.bot = bot
         self.interaction = interaction
         self.command = None
+        self._state: ConnectionState = self.interaction._state
+
+    async def _get_channel(self) -> discord.abc.Messageable:
+        return self.channel
 
     @cached_property
     def channel(self):
@@ -87,9 +93,6 @@ def user(self) -> Optional[Union[Member, User]]:
     def voice_client(self):
         return self.guild.voice_client
 
-    def typing(self):
-        return Typing(self.channel)
-
     @cached_property
     def response(self) -> InteractionResponse:
         return self.interaction.response
@@ -100,11 +103,6 @@ def response(self) -> InteractionResponse:
     def respond(self):
         return self.followup.send if self.response.is_done() else self.interaction.response.send_message
 
-    @property
-    def send(self):
-        """Behaves like :attr:`~discord.abc.Messagable.send` if the response is done, else behaves like :attr:`~discord.app.ApplicationContext.respond`"""
-        return self.channel.send if self.response.is_done() else self.respond
-
     @property
     def defer(self):
         return self.interaction.response.defer

discord/bot.py

@@ -234,7 +234,7 @@ async def process_application_commands(self, interaction: Interaction) -> None:
             else:
                 self.dispatch('application_command_completion', ctx)
 
-    def slash_command(self, **kwargs) -> SlashCommand:
+    def slash_command(self, **kwargs):
         """A shortcut decorator that invokes :func:`.ApplicationCommandMixin.command` and adds it to
         the internal command list via :meth:`~.ApplicationCommandMixin.add_application_command`.
         This shortcut is made specifically for :class:`.SlashCommand`.
@@ -249,7 +249,7 @@ def slash_command(self, **kwargs) -> SlashCommand:
         """
         return self.application_command(cls=SlashCommand, **kwargs)
 
-    def user_command(self, **kwargs) -> UserCommand:
+    def user_command(self, **kwargs):
         """A shortcut decorator that invokes :func:`.ApplicationCommandMixin.command` and adds it to
         the internal command list via :meth:`~.ApplicationCommandMixin.add_application_command`.
         This shortcut is made specifically for :class:`.UserCommand`.
@@ -264,7 +264,7 @@ def user_command(self, **kwargs) -> UserCommand:
         """
         return self.application_command(cls=UserCommand, **kwargs)
 
-    def message_command(self, **kwargs) -> MessageCommand:
+    def message_command(self, **kwargs):
         """A shortcut decorator that invokes :func:`.ApplicationCommandMixin.command` and adds it to
         the internal command list via :meth:`~.ApplicationCommandMixin.add_application_command`.
         This shortcut is made specifically for :class:`.MessageCommand`.

discord/channel.py

@@ -45,7 +45,7 @@
 
 import discord.abc
 from .permissions import PermissionOverwrite, Permissions
-from .enums import ChannelType, StagePrivacyLevel, try_enum, VoiceRegion, VideoQualityMode
+from .enums import ChannelType, InviteTarget, StagePrivacyLevel, try_enum, VoiceRegion, VideoQualityMode
 from .mixins import Hashable
 from .object import Object
 from . import utils
@@ -55,6 +55,7 @@
 from .stage_instance import StageInstance
 from .threads import Thread
 from .iterators import ArchivedThreadIterator
+from .invite import Invite
 
 __all__ = (
     'TextChannel',
@@ -1037,6 +1038,64 @@ async def edit(self, *, reason=None, **options):
             # the payload will always be the proper channel payload
             return self.__class__(state=self._state, guild=self.guild, data=payload)  # type: ignore
 
+    async def create_activity_invite(self, event:str, **kwargs) -> Invite:
+        """|coro|
+
+        A shortcut method that creates an instant activity invite.
+
+        You must have the :attr:`~discord.Permissions.create_instant_invite` permission to
+        do this.
+
+        Parameters
+        ------------
+        event: :class:`str`
+            The event to create an invite for. 
+        max_age: :class:`int`
+            How long the invite should last in seconds. If it's 0 then the invite
+            doesn't expire. Defaults to ``0``.
+        max_uses: :class:`int`
+            How many uses the invite could be used for. If it's 0 then there
+            are unlimited uses. Defaults to ``0``.
+        temporary: :class:`bool`
+            Denotes that the invite grants temporary membership
+            (i.e. they get kicked after they disconnect). Defaults to ``False``.
+        unique: :class:`bool`
+            Indicates if a unique invite URL should be created. Defaults to True.
+            If this is set to ``False`` then it will return a previously created
+            invite.
+        reason: Optional[:class:`str`]
+            The reason for creating this invite. Shows up on the audit log.
+
+
+        Raises
+        -------
+        InvalidArgument
+            If the event is not a valid event.
+        ~discord.HTTPException
+            Invite creation failed.
+
+        Returns
+        --------
+        :class:`~discord.Invite`
+            The invite that was created.
+        """
+
+        application_ids = {
+            'youtube' : 755600276941176913,
+            'poker'   : 755827207812677713,
+            'betrayal': 773336526917861400,
+            'fishing' : 814288819477020702,
+            'chess'   : 832012774040141894,
+        }
+        event = application_ids.get(event)
+        if event is None:
+            raise InvalidArgument('Invalid event.')
+
+        return await self.create_invite(
+            target_type=InviteTarget.embedded_application,
+            target_application_id=event,
+            **kwargs
+        )
 
 class StageChannel(VocalGuildChannel):
     """Represents a Discord guild stage channel.

discord/client.py

@@ -460,13 +460,17 @@ async def login(self, token: str) -> None:
 
         Raises
         ------
+        TypeError
+            The token was in invalid type.
         :exc:`.LoginFailure`
             The wrong credentials are passed.
         :exc:`.HTTPException`
             An unknown HTTP related error occurred,
             usually when it isn't 200 or the known incorrect credentials
             passing status code.
         """
+        if not isinstance(token, str):
+            raise TypeError(f"token must be of type str, not {token.__class__.__name__}")
 
         _log.info('logging in using static token')
 

discord/colour.py

@@ -325,5 +325,13 @@ def yellow(cls: Type[CT]) -> CT:
         """
         return cls(0xFEE75C)
 
+    @classmethod
+    def nitro_pink(cls: Type[CT]) -> CT:
+        """A factory method that returns a :class:`Colour` with a value of ``0xf47fff``.
+
+        .. versionadded:: 2.0
+        """
+        return cls(0xf47fff)
+
 
 Color = Colour