Skip to content

feat: components v2 & View improvements #2707

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 243 commits into
base: master
Choose a base branch
from

Conversation

NeloBlivion
Copy link
Member

@NeloBlivion NeloBlivion commented Feb 6, 2025

Summary

Soon




Information

  • This PR fixes an issue.
  • This PR adds something new (e.g. new method or parameters).
  • This PR is a breaking change (e.g. methods or parameters removed/renamed).
  • This PR is not a code change (e.g. documentation, README, typehinting,
    examples, ...).

Checklist

  • I have searched the open pull requests for duplicates.
  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes.
  • If type: ignore comments were used, a comment is also left explaining why.
  • I have updated the changelog to include these changes.

Reference:

message-components-v2-text-display
message-components-v2-container
message-components-v2-file
message-components-v2-media-gallery
message-components-v2-section
message-components-v2-separator

@Lulalaby Lulalaby added discord limitation Limitation imposed by discord priority: medium Medium Priority status: in progress Work in Progess feature Implements a feature upcoming discord feature Involves a feature not yet fully released by Discord labels Feb 6, 2025
@Lulalaby Lulalaby added this to the v2.7 milestone Feb 6, 2025
@Lulalaby
Copy link
Member

discord/discord-api-spec@2a2605c

we have attachment_id on the UnfurledMedia object now

@Lulalaby
Copy link
Member

and the file display component has two more fields: size in bytes and name
ref discord/discord-api-docs#7609

NeloBlivion and others added 25 commits June 11, 2025 02:57
Co-authored-by: plun1331 <[email protected]>
Co-authored-by: JustaSqu1d <[email protected]>
Signed-off-by: UK <[email protected]>
Co-authored-by: Dorukyum <[email protected]>
Co-authored-by: JustaSqu1d <[email protected]>
Signed-off-by: UK <[email protected]>
@Paillat-dev
Copy link
Contributor

@NeloBlivion I think something broke with your recent pushes:

if payload["content"] or payload["embeds"]

in ``discord/webhooks/_async.py`

Is causing:

KeyError: 'content'
  File "discord/commands/core.py", line 138, in wrapped
    ret = await coro(arg)
  File "discord/commands/core.py", line 1078, in _invoke
    await self.callback(self.cog, ctx, **kwargs)
  File "src/extensions/chatcop/cogs/admin/__init__.py", line 260, in view
    await ctx.respond(view=view, ephemeral=True, allowed_mentions=discord.AllowedMentions.none())
  File "discord/ext/bridge/context.py", line 92, in respond
    return await self._respond(*args, **kwargs)
  File "discord/ext/bridge/context.py", line 145, in _respond
    return await self._get_super("respond")(*args, **kwargs)
  File "discord/interactions.py", line 694, in respond
    return await self.followup.send(*args, **kwargs)
  File "discord/webhook/async_.py", line 1822, in send
    params = handle_message_parameters(
  File "discord/webhook/async_.py", line 673, in handle_message_parameters
    if payload["content"] or payload["embeds"]:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discord limitation Limitation imposed by discord feature Implements a feature hold: changelog This pull request is missing a changelog entry priority: medium Medium Priority status: in progress Work in Progess upcoming discord feature Involves a feature not yet fully released by Discord
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.