Repository navigation
Conversation
Migrate custom colour handling and embed construction to pointer-based colour values, preserving nil colours and guild-specific defaults across commands, tickets, panels, and welcome messages. Enable local database and GDL module replacements for development.
This PR is blocked
Remove the label(s) above once the underlying dependency bump or comment has landed, and this check will re-run on the next push. |
This file contains hidden or 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
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.
RM-348
Description
Colours are
*intthroughout: nil sends no colour (Discord's default bar), and0sends black.GetColours,GetColourOrDefault,GetColourForGuildandReplyable.GetColourreturn*int, and aNULLcustom colour means no colour. Default colour maps are copied instead of shared.BuildEmbed/BuildEmbedRaw, welcome and tag embeds, form answers, /switchpanel, the close embed, /setup, help and the exit survey all send the colour throughSetColorPtr.accent_color, so no colour gives Discord's default bar instead of a black one.NULL= no colour,0= black). Panels without settings still use the guild Failure colour.customisation.GetColour.Needs:
TicketsBot-cloud/gdl#19
TicketsBot-cloud/database#49
Type of Change
Testing
color/accent_color, and0sends"color":0.Checklist