Skip to content

feat: add support for limited partner colors and force refresh of guilds - #562

Merged
stijnvdkolk merged 10 commits into
mainfrom
feature/559-restrict-partner-colors-by-server-membership
May 19, 2026
Merged

feat: add support for limited partner colors and force refresh of guilds#562
stijnvdkolk merged 10 commits into
mainfrom
feature/559-restrict-partner-colors-by-server-membership

Conversation

@stijnvdkolk

@stijnvdkolk stijnvdkolk commented May 17, 2026

Copy link
Copy Markdown
Contributor

If invite link is present:
image

If no invite link is present:
image

Clicking on refresh shows this inline (temporary makes it scrollable):
image

Added force refresh button on me page:
image

@stijnvdkolk stijnvdkolk added backend An issue related to our backend package frontend An issue related to our frontend package labels May 17, 2026
@stijnvdkolk stijnvdkolk linked an issue May 17, 2026 that may be closed by this pull request
@stijnvdkolk
stijnvdkolk force-pushed the feature/559-restrict-partner-colors-by-server-membership branch 2 times, most recently from ef9d54b to 6bb405e Compare May 17, 2026 22:01
@stijnvdkolk
stijnvdkolk marked this pull request as ready for review May 17, 2026 22:04
@stijnvdkolk
stijnvdkolk requested review from Rocked03 and jaskfla and removed request for Rocked03 May 17, 2026 22:04
Base automatically changed from feature/490-move-all_colors_global-from-backend-env-to-database-value to main May 18, 2026 06:36
@stijnvdkolk
stijnvdkolk force-pushed the feature/559-restrict-partner-colors-by-server-membership branch 3 times, most recently from 5d08d25 to d562016 Compare May 18, 2026 07:22

@jaskfla jaskfla left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Like it overall! Couple nitpicks details from UX/UI perspective, but arguably commensurate with your attentiveness to detail

Great addition, and appreciate the screenshots in the PR description 🙏

Comment thread packages/backend/src/routes/api/v1/discord.ts
Comment thread packages/backend/src/services/discordGuildService.test.ts Outdated
Comment thread packages/backend/src/services/discordGuildService.ts Outdated
Comment thread packages/backend/src/services/discordGuildService.ts Outdated
Comment thread packages/backend/src/services/pixelService.ts
Comment thread packages/frontend/src/components/RecheckMemberships.tsx Outdated
Comment thread packages/frontend/src/components/RecheckMemberships.tsx
Comment thread packages/frontend/src/components/action-panel/tabs/SelectedColorInfoCard.tsx Outdated
Comment thread packages/frontend/src/components/swatch/InteractiveSwatch.tsx Outdated
Comment thread packages/frontend/src/components/swatch/InteractiveSwatch.tsx Outdated
Comment thread packages/frontend/src/components/swatch/InteractiveSwatch.tsx Outdated
@stijnvdkolk
stijnvdkolk force-pushed the feature/559-restrict-partner-colors-by-server-membership branch from 7110b8e to 730e2ea Compare May 18, 2026 22:02
@stijnvdkolk
stijnvdkolk requested review from Rocked03 and jaskfla May 18, 2026 22:02
@stijnvdkolk
stijnvdkolk force-pushed the feature/559-restrict-partner-colors-by-server-membership branch from 3a4031a to 788d708 Compare May 19, 2026 21:36

@jaskfla jaskfla left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Couple optional details, one request with the fancy luminance logic, but pre-approving ✅

Thanks Stijn 🙏

Comment thread packages/backend/src/services/discordGuildService.ts
Comment thread packages/frontend/src/components/action-panel/tabs/SelectedColorInfoCard.tsx Outdated
Comment thread packages/frontend/src/components/swatch/StaticSwatch.tsx Outdated
…ontrast-color for StaticSwatch styling instead of custom luminance logic
@stijnvdkolk
stijnvdkolk enabled auto-merge May 19, 2026 23:10
@stijnvdkolk
stijnvdkolk added this pull request to the merge queue May 19, 2026
Merged via the queue into main with commit 1c5f99b May 19, 2026
14 checks passed
@stijnvdkolk
stijnvdkolk deleted the feature/559-restrict-partner-colors-by-server-membership branch May 19, 2026 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend An issue related to our backend package frontend An issue related to our frontend package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Restrict partner colors by server membership

4 participants