Skip to content

feat!: use the guild data provided on interactions #1236

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

feat!: use the guild data provided on interactions #1236

wants to merge 2 commits into from
+81 −21

Conversation

onerandomusername
Copy link
Member

@onerandomusername onerandomusername commented Sep 23, 2024 •
edited
Loading

This could in theory use an optimisation of caching the guild, but as we do not have a good way to clear the cache afterwards, this is the best we can implement at this point in time.

Summary

Checklist

  • If code changes were made, then they have been tested
    • I have updated the documentation to reflect the changes
    • I have formatted the code properly by running pdm lint
    • I have type-checked the code by running pdm pyright
  • 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, ...)

Replacement for #647

@onerandomusername onerandomusername added the feature request Request for a new feature label Sep 23, 2024
@onerandomusername onerandomusername marked this pull request as draft September 23, 2024 08:33
@onerandomusername onerandomusername force-pushed the arielle/always-provide-guild-for-guild-interactions branch from 40bcbe1 to ceac6eb Compare September 23, 2024 08:38
@shiftinv shiftinv added t: enhancement New feature t: api support Support of Discord API features s: in progress Issue/PR is being worked on and removed feature request Request for a new feature labels Sep 23, 2024
@shiftinv shiftinv mentioned this pull request Oct 1, 2024
Merged
8 tasks
@shiftinv shiftinv added this to the disnake v2.11 milestone Nov 29, 2024
@shiftinv shiftinv mentioned this pull request Mar 6, 2025
Merged
8 tasks
@shiftinv shiftinv removed this from the disnake v2.11 milestone Aug 24, 2025
@onerandomusername
feat!: use the guild data provided on interactions
0ff1f62 
This could in theory use an optimisation of caching the guild, but  as
we do not have a good way to clear the cache afterwards, this is the
best we can implement at this point in time.
@onerandomusername onerandomusername force-pushed the arielle/always-provide-guild-for-guild-interactions branch from da11276 to 0ff1f62 Compare September 9, 2025 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shiftinv shiftinv Awaiting requested review from shiftinv

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

s: in progress Issue/PR is being worked on t: api support Support of Discord API features t: enhancement New feature

Projects

disnake
Status: Todo

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@onerandomusername @shiftinv