Skip to content

RM-348: Add No colour option for embeds - #162

Open
biast12 wants to merge 2 commits into
masterfrom
RM-348
Open

biast12 wants to merge 2 commits into
masterfrom
RM-348

Conversation

@biast12

@biast12 biast12 commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

RM-348

Description

Every embed colour picker gets an × that sets the colour to No colour (Discord's default bar). The API uses null for no colour, and 0 / #000000 is black.

  • Pickers: panel, welcome message, multi-panel, tag, KB article, setup wizard, out-of-hours, and Success/Failure. ColourSelect gets an opt-in nullable / clearable variant.
  • Previews draw Discord's default border for no colour, and panel colour dots are hidden.
  • Backend: colour fields are pointers (null ↔ NULL). Panel and out-of-hours colours are range-checked, and settings colours accept null. Panel, multi-panel and gallery payloads leave color out for no colour.
  • Also fixed:
    • the KB editor no longer turns a missing colour into blurple;
    • the setup wizard no longer turns black into blurple;
    • GalleryCard no longer crashes on a null colour;
    • transcripts now show black embeds and container accents.

Needs:
TicketsBot-cloud/gdl#19
TicketsBot-cloud/database#49
TicketsBot-cloud/worker#162

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Improvement

Checklist

  • My code follows the style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Allow panel, gallery, support-hours, custom embed, and guild setting colours to be cleared and preserved as null across the Go API and React UI. Add shared colour conversion and validation helpers, update previews and selectors to handle missing colours, and enable local backend module replacements.
@BenHall-1

Copy link
Copy Markdown
Member

Ready for merge on db & gdl and bump :)

This branch has not been deployed

No deployments
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.

2 participants