Skip to content

docs(AuditLogKey): correct expected type for MEMBER_ROLES_ADD/REMOVE#2940

Merged
MinnDevelopment merged 3 commits into
discord-jda:masterfrom
Dking08:fix-AuditLogKey-expected-type
Nov 2, 2025
Merged

docs(AuditLogKey): correct expected type for MEMBER_ROLES_ADD/REMOVE#2940
MinnDevelopment merged 3 commits into
discord-jda:masterfrom
Dking08:fix-AuditLogKey-expected-type

Conversation

@Dking08

@Dking08 Dking08 commented Oct 21, 2025

Copy link
Copy Markdown
Contributor

Update documentation for MEMBER_ROLES_ADD and MEMBER_ROLES_REMOVE to reflect new expected data structure (#2825)

Pull Request Etiquette

Changes

  • Documentation

Closes Issue: #2825

Description

Correct the Javadoc for AuditLogKey.MEMBER_ROLES_ADD and AuditLogKey.MEMBER_ROLES_REMOVE to reflect the actual payload shape returned by Discord audit logs.

Before: Expected type documented as List<String> (role IDs).
After: Expected type documented as List<Map<String, String>> with entries for id and name.

Documentation-only change; no API or behavior changes.

@Dking08

Dking08 commented Oct 27, 2025

Copy link
Copy Markdown
Contributor Author

@MinnDevelopment Looking forward to your feedback.

@MinnDevelopment MinnDevelopment added this pull request to the merge queue Nov 2, 2025
Merged via the queue into discord-jda:master with commit 26d1fbc Nov 2, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants