refactor: SKU.subscriptions#user should take a snowflake, not int

disnake/iterators.py

@@ -1155,8 +1155,8 @@ def __init__(
         self,
         sku_id: int,
         *,
-        user_id: Optional[int] = None,
         state: ConnectionState,
+        user_id: Optional[int] = None,  # required, except for oauth queries
         limit: Optional[int] = None,
         before: Optional[Union[Snowflake, datetime.datetime]] = None,
         after: Optional[Union[Snowflake, datetime.datetime]] = None,

disnake/sku.py

@@ -13,7 +13,7 @@
 from .utils import snowflake_time
 
 if TYPE_CHECKING:
-    from .abc import SnowflakeTime
+    from .abc import Snowflake, SnowflakeTime
     from .state import ConnectionState
     from .types.sku import SKU as SKUPayload
 
@@ -89,21 +89,21 @@ def flags(self) -> SKUFlags:
 
     async def subscriptions(
         self,
-        user_id: int,
+        user: Snowflake,
         *,
         limit: Optional[int] = 50,
         before: Optional[SnowflakeTime] = None,
         after: Optional[SnowflakeTime] = None,
     ) -> SubscriptionIterator:
         """|coro|
 
-        Retrieves an :class:`.AsyncIterator` that enabled receiving subscriptions for the SKU.
+        Retrieves an :class:`.AsyncIterator` that enables receiving subscriptions for the SKU.
 
-        All parameters are optional.
+        All parameters, except ``user``, are optional.
 
         Parameters
         ----------
-        user: :class:`int`
+        user: :class:`abc.Snowflake`
             The user to retrieve subscriptions for.
         limit: Optional[:class:`int`]
             The number of subscriptions to retrieve.
@@ -132,10 +132,10 @@ async def subscriptions(
         return SubscriptionIterator(
             self.id,
             state=self._state,
+            user_id=user.id,
             limit=limit,
             before=before,
             after=after,
-            user_id=user_id,
         )
 
     async def fetch_subscription(self, subscription_id: int, /) -> Subscription: