-
Notifications
You must be signed in to change notification settings - Fork 415
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
fix(getGuildAuditLog): cache users & threads earlier #1367
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DonovanDMC
added a commit
to DonovanArchive/ErisPRUpdateBot
that referenced
this pull request
May 7, 2022
- rename "main" to "upstream" - add "everything-v10" branch Additions: - abalabahaha/eris#1283 - abalabahaha/eris#1285 - abalabahaha/eris#1332 - abalabahaha/eris#1344 - abalabahaha/eris#1351 - abalabahaha/eris#1355 - abalabahaha/eris#1358 - abalabahaha/eris#1359 - abalabahaha/eris#1363 - abalabahaha/eris#1366 - abalabahaha/eris#1367 - abalabahaha/eris#1368 - abalabahaha/eris#1369 - abalabahaha/eris#1370 Removals: - abalabahaha/eris#1305 - abalabahaha/eris#1314 - abalabahaha/eris#1325
|
abalabahaha
approved these changes
Jun 5, 2022
DonovanDMC
added a commit
to DonovanArchive/ErisPRUpdateBot
that referenced
this pull request
Jun 8, 2022
Additions: - abalabahaha/eris#1275 - abalabahaha/eris#1309 (technically) - abalabahaha/eris#1379 - abalabahaha/eris#1384 Removals: - abalabahaha/eris#1328 - abalabahaha/eris#1332 -abalabahaha/eris# 1335 - abalabahaha/eris#1344 - abalabahaha/eris#1363 - abalabahaha/eris#1367 - abalabahaha/eris#1370 - abalabahaha/eris#1376 - remove `interactions` branch - remove unused remotes
Brainicism
added a commit
to Brainicism/eris
that referenced
this pull request
Feb 13, 2023
* fix(voice): re-add receive listener on connect (abalabahaha#1332) * fix(getAllUsers): request presences if intent is enabled (abalabahaha#1378) * chore(Base): document props, static methods (abalabahaha#1380) * fix(Guild#editCommandPermissions): fix alias typo (abalabahaha#1350) * fix(getGuildAuditLog): cache users & threads earlier (abalabahaha#1367) * fix(messageUpdate): fix oldMessage#mentions type (abalabahaha#1377) * fix(joinVoiceChannel): skip permission check if Member uncached (abalabahaha#1383) Co-authored-by: Donovan Daniels <hewwo@yiff.rocks> * fix(ShardManager): refactor concurrency check, add option (abalabahaha#1382) * fix(createRole): split options into separate overloads (abalabahaha#1376) * feat(getGuildBans): support pagination (abalabahaha#1363) Co-authored-by: abalabahaha <hi@abal.moe> * feat(editChannelPositions): add support for endpoint (abalabahaha#1328) Co-authored-by: bsian03 <chharry321@gmail.com> Co-authored-by: abalabahaha <hi@abal.moe> * fix(editChannel): fix `videoQualityMode` & add `permissionOverwrites` (abalabahaha#1319) * feat(connect): add token check (abalabahaha#1283) * feat(Client): allow custom numbers in intents option (abalabahaha#1388) Co-authored-by: Donovan Daniels <hewwo@yiff.rocks> Co-authored-by: abalabahaha <hi@abal.moe> * feat(interactions): Add support for appPermissions (abalabahaha#1394) * fix(Client): remove optional chaining from 05a932a * 0.17.1 * chore: bump dev version to 0.17.2-dev * feat(Member): Add dynamicAvatarURL (abalabahaha#1411) * feat(permissions): Slash Permissions v2 (PR 1372) * Attachment type (PR 1358) * Remove all user account features (PR 1338) * refactor(interactions): Interactions Refactor (PR 1309) * feat(interactions): Support Modals (PR 1404) * Use process warnings for deprecations (PR 1276) * Guild Scheduled Events (PR 1275) * Follow http/https redirects when playing song (PR 494) * Return void on 204 (PR 1021) * Allow configuring RequestHandler to use HTTP instead of HTTPS (PR 1193) * Standardized parsing errors (PR 1227) * feat(gateway): Use New Resume URL (PR 1410) * refactor: remove StoreChannels (PR 1405) * feat(bans): Seconds Support (PR 1397) * feat(AutoModeration): Support Auto Moderation (PR 1390) * feat(options): Gateway Options (PR 1384) * refactor: Overall Consistency Fixes & Minor Feature Additions (PR 1379) * fix(CommandClient): await permission checks (PR 1414) * feat: API v10 (PR 1371) * chore: rebrand to Project Dysnomia * lint * fix: use ./* for exports * chore: rename forgotten references to Eris (#2) For the time being, until I figure out what to do with references to Eris' website, appropriate links in code/Discord's API docs have been linked. * chore: remove deprecated/dead code (#3) * feat: entity select types (#4) * chore(examples/intent): fix import typo (#6) * chore: update example in README.md (#7) * feat: modern file uploading (#5) * fixup(Client): oops :( make _processAttachments return the expected structure (#8) Also do not pass empty file arrays (triggers data being sent over as multipart data) * feat(automoderation): support new trigger metadata (#9) * support new trigger metadata types * update trigger type constants * deprecate camel-cased trigger metadata, use raw metadata instead * feat(Constants): add ACTIVE_DEVELOPER user flag (#10) * feat: forum support (#1) I believe this should be enough to ensure full forum support. Although, some things might've been missed out by accident. * Attachment#proxyUrl -> Attachment#proxyURL (#11) * fix(Client): do not send the file buffer twice (#13) * feat(Guild): automatically populate events on GUILD_CREATE (#12) * feat(ForumChannel): `default_forum_layout` (#14) * chore: update dev dependencies and lint (#15) * chore: remove editGuildVanity (abalabahaha#18) * Removed editGuildVanity The vanity endpoint was disabled for bots ``` { "message": "Bots cannot use this endpoint", "code": 20001 } ``` * remove Guild#editVanity * update TSDs appropriately Co-authored-by: Lars_und_so <46791248+Larsundso@users.noreply.github.com> * feat(ApplicationCommand): support NSFW commands (abalabahaha#16) * docs: document audit log after parameter (abalabahaha#20) ref: discord/discord-api-docs@38f01b0 * fix(Constants): wrong ACTIVE_DEVELOPER bit (╯°□°)╯︵ ┻━┻ (abalabahaha#23) * feat: add new message types (abalabahaha#22) * feat: add new message types currently won't apply any transformations to message content, as I don't know how the messages look like. ref: discord/discord-api-docs@e9711be ref: discord/discord-api-docs@83f6c85 * don't do anything special to the messages right now * feat: add GIF as a supported sticker format (abalabahaha#24) ref: discord/discord-api-docs@5214f12 * feat: linked roles (abalabahaha#19) * add role connection verification url into OAuthAppInfo * Add Client#{edit,get}RoleConnectionMetadata Interface may be subject to change * add guild connection role tag * add role connection metadata type enum * add types * feat(ThreadChannel): thread member pagination (abalabahaha#25) * feat(ThreadChannel): allow fetching more details about thread members * feat(ThreadChannel): add singular thread member fetching * remove redundant parameter * member data from THREAD_MEMBERS_UPDATE is sent in the member field * handle the guild not being cached accordingly * consistently transform options in getThreadMember * feat: role subscriptions (abalabahaha#21) * basic role subscription implementation * add role subscription data to Message object see API docs PR 5828 * align guild features with API docs * remove TODO comment roleSubscriptionData will be kept as a camelCased object * fix(ThreadMember): missing Member import (abalabahaha#27) * feat(Client): add threadID support to editWebhookMessage (abalabahaha#28) * fix(Client): add threadID support to editWebhookMessage * reword doc for threadID * add threadID doc to Message#editWebhook * update ts definitions * fix(ShardManager): correctly emit warning on invalid intent (abalabahaha#29) * fix(ForumChannel): document and handle nullability of certain fields --------- Co-authored-by: Jim Chen <10269287+jimchen5209@users.noreply.github.com> Co-authored-by: EXPLOSION <git@helvetica.moe> Co-authored-by: Erin <github@eritbh.me> Co-authored-by: Chris <67835702+NotChris123@users.noreply.github.com> Co-authored-by: Donovan Daniels <hewwo@yiff.rocks> Co-authored-by: abal <hi@abal.moe> Co-authored-by: HeadTriXz <32986761+HeadTriXz@users.noreply.github.com> Co-authored-by: Carlos Lima <xaxim@users.noreply.github.com> Co-authored-by: bsian03 <chharry321@gmail.com> Co-authored-by: isaiah <662614+aicushman@users.noreply.github.com> Co-authored-by: Catboy <50190590+Catboi8@users.noreply.github.com> Co-authored-by: HcgRandon <me@randon.moe> Co-authored-by: F Robinson <freddy@protality.plus.com> Co-authored-by: Alex <alex.dumbreck@gmail.com> Co-authored-by: Loliticos <loliticos@outlook.com> Co-authored-by: macdja38 <jakeincanada@icloud.com> Co-authored-by: Thijs Molendijk <thijs.molendijk@gmail.com> Co-authored-by: LJNeon <23249107+LJNeon@users.noreply.github.com> Co-authored-by: yalpha <44028028+YalphaFR@users.noreply.github.com> Co-authored-by: TTtie <me@tttie.cz> Co-authored-by: Sham <thesham@protonmail.com> Co-authored-by: Lars_und_so <46791248+Larsundso@users.noreply.github.com> Co-authored-by: Cynthia Foxwell <LUModder@gmail.com> Co-authored-by: Snazzah <7025343+Snazzah@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently, users are added to the cache too late when audit log entries are fetched, so they will not be present in the entry itself if the user was not previously cached. This pull request remedies that by pulling it out of the object itself, and making it before the entries are constructed.