Skip to content

Added missing guild role fields#286

Merged
BrammyS merged 8 commits intodevfrom
features/285-missing-fields-on-guild-role
Jul 16, 2025
Merged

Added missing guild role fields#286
BrammyS merged 8 commits intodevfrom
features/285-missing-fields-on-guild-role

Conversation

@BrammyS
Copy link
Member

@BrammyS BrammyS commented Jul 16, 2025

Description

Marks DiscordGuildRoleData.Color as obsolote because it is now replaced by colors.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@BrammyS BrammyS requested a review from Copilot July 16, 2025 13:31
@BrammyS BrammyS self-assigned this Jul 16, 2025

This comment was marked as outdated.

BrammyS and others added 3 commits July 16, 2025 15:33
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@BrammyS BrammyS requested a review from Copilot July 16, 2025 13:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds missing guild role fields to support newer Discord API features. The changes expand the guild role model to include additional properties like colors, icons, tags, and flags that were previously missing from the implementation.

  • Adds new role color system with primary, secondary, and tertiary colors
  • Implements role tags structure for bot roles, integrations, and premium features
  • Adds role flags enum and icon/emoji support

Reviewed Changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
DiscordGuildRoleData.cs Extended with new properties for icons, colors, tags, and flags
DiscordGuildRole.cs Updated constructor and properties to handle new role fields
DiscordGuildRoleColorsData.cs New data model for role color structure
DiscordGuildRoleTagsData.cs New data model for role tags with custom converter
DiscordGuildRoleFlags.cs New enum for role flags
Various test JSON files Updated test data to include new role properties

@BrammyS BrammyS merged commit af2a6c1 into dev Jul 16, 2025
1 check passed
@BrammyS BrammyS deleted the features/285-missing-fields-on-guild-role branch July 16, 2025 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant