Visitar URL original
AuditlogParser update by Plerx2493 · Pull Request #1612 · DSharpPlus/DSharpPlus · GitHub
Skip to content

AuditlogParser update - #1612

Merged
akiraveliara merged 10 commits into
DSharpPlus:masterfrom
Plerx2493:fix/auditKeys
Sep 12, 2023
Merged

akiraveliara merged 10 commits into
DSharpPlus:masterfrom
Plerx2493:fix/auditKeys

Conversation

@Plerx2493

@Plerx2493 Plerx2493 commented Sep 3, 2023 •

Copy link
Copy Markdown
Member

Summary

Notes

https://discord.com/channels/379378609942560770/379386901725052928/1147974237735366818

@Plerx2493 Plerx2493 added enhancement audit-log Issues about missing auditlog types or missing change key labels Sep 3, 2023
@Plerx2493 Plerx2493 changed the title Add missing flags and userLimit changes AuditlogParser update Sep 3, 2023
@Plerx2493
Plerx2493 marked this pull request as draft September 4, 2023 11:30
@Plerx2493
Plerx2493 marked this pull request as ready for review September 12, 2023 15:53

@akiraveliara akiraveliara left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

three naming issues, other than that lgtm

@akiraveliara
akiraveliara self-requested a review September 12, 2023 16:43
@akiraveliara
akiraveliara merged commit ef4949f into DSharpPlus:master Sep 12, 2023
@Plerx2493
Plerx2493 deleted the fix/auditKeys branch September 12, 2023 16:43
OoLunar pushed a commit that referenced this pull request Oct 3, 2024
* Add missing flags and userLimit changes

* fix casting

* AuditLogParser cleanup

* Added missing keys and refactoring

* remove debug/testing stuff

* IAsyncEnumerable

* fix #1569

* add missing property change on channel entries

* rename some method parameters to follow editorconfig

* revert misclick
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

audit-log Issues about missing auditlog types or missing change key enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DiscordAuditLogMessageEntry.Target property is of wrong type.

2 participants