Skip to content

Add submission blacklist - #153

Merged
BenHall-1 merged 3 commits into
masterfrom
submission-blacklist
Oct 2, 2026
Merged

BenHall-1 merged 3 commits into
masterfrom
submission-blacklist

Conversation

@biast12

@biast12 biast12 commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Adds admin blacklists that stop users and servers from submitting to the gallery, and users from the public integration review queue. Also limits who can remove blacklist entries and manage integrations.

Blacklists

  • New Admin pages:
    • Gallery Blacklist: users and servers; admin and owner.
    • Integration Blacklist: users only; owner only.
  • Gallery: a listed user, or anyone submitting from a listed server, gets a 403 on submit, submit-tag, submit-form and resubmit.
  • Integrations: a listed user gets a 403 when requesting public review, or when editing an integration that's public or pending review. Create, activate and delete are unaffected.

Permissions (narrowed for admins)

  • Only the owner can remove entries, on all four blacklists (Gallery, Integration, Server, Global). Admins can still add to the Gallery and Server blacklists.
  • Admin Integrations (list, detail, approve, reject, unapprove) and the "Integration Requests" notification are now owner-only.

UI

  • Blacklist cards on all blacklist pages show an avatar or icon, a name and the ID, with fallbacks: default avatar, server-icon tile, "Unknown User"/"Unknown Server". The Server Blacklist can now be searched by server name.
  • The grid is now 2 columns from lg and 3 from xl. Admin tabs grow to fit longer labels.
  • Server names come from a new batch cache.GetGuilds. If the cache fails, the Server Blacklist returns entries without names instead of a 500, because AdminLayout's access check calls that route. The gallery and integration lists still return 500.
  • Fix: faTrash and faSync weren't registered. So the trash icons (blacklists, SKUs, Polar products) and the Servers refresh icon never rendered, and the blacklist remove buttons, which are icon-only, were empty.

REQUIRED: TicketsBot-cloud/database#46

Type of Change

  • Bug fix
  • New feature
  • Breaking change

Testing

  • Blacklist a user and server for gallery and integration submission
  • Try to publish a panel, form, integration with that user and server and it should fail

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

Adds gallery and integration submission blacklist APIs, database enforcement, audit logging, and admin UI management. Includes user/server metadata resolution, blacklist avatar rendering, owner-only integration controls, and updated admin access permissions.
@biast12
biast12 requested a review from a team as a code owner September 24, 2026 23:13
@github-actions github-actions Bot added type:bug Bug Fixes type:feature New Feature labels Sep 24, 2026
Update admin navigation links to use flex centering and a consistent responsive width, improving tab alignment across screen sizes.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Deploying tickets-v2-dashboard with  Cloudflare Pages  Cloudflare Pages

Latest commit: a83c9cf
Status: ✅  Deploy successful!
Preview URL: https://e93d72c1.tickets-v2-dashboard.pages.dev
Branch Preview URL: https://submission-blacklist.tickets-v2-dashboard.pages.dev

View logs

@BenHall-1

Copy link
Copy Markdown
Member

Readyfor bump!

@biast12

biast12 commented Sep 27, 2026

Copy link
Copy Markdown
Contributor Author

@BenHall-1 bumped!!

@BenHall-1
BenHall-1 merged commit 4965d0c into master Oct 2, 2026
2 checks passed
@BenHall-1
BenHall-1 deleted the submission-blacklist branch October 2, 2026 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Bug Fixes type:feature New Feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants