Skip to content

feat(admin): make suspicious domains configurable#3562

Merged
markijbema merged 1 commit into
mainfrom
mark/bookmark-blacklisted-domain-tabs
May 28, 2026
Merged

feat(admin): make suspicious domains configurable#3562
markijbema merged 1 commit into
mainfrom
mark/bookmark-blacklisted-domain-tabs

Conversation

@markijbema

@markijbema markijbema commented May 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds bookmarkable URLs for each Blacklisted Domains admin tab via ?tab=edit|stats|suspicious.
  • Makes the Suspicious tab's high-volume provider noise filter configurable with a default-on checkbox.
  • Keeps the existing 30% blocked-account threshold as the default server-side behavior while allowing admins to disable it.

Verification

  • Not manually verified in browser.

Visual Changes

CleanShot 2026-05-28 at 16 10 57

Reviewer Notes

  • The Suspicious query still defaults to hiding high-volume providers when called without input.

@markijbema markijbema marked this pull request as ready for review May 28, 2026 14:11
@markijbema markijbema enabled auto-merge (squash) May 28, 2026 14:13
@kilo-code-bot

kilo-code-bot Bot commented May 28, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

All three changed files are correct — the optional input schema with conditional .having() filter, the checkbox toggle with URL-based tab navigation, and the new test case are all sound with no security, runtime, or logic issues.

Files Reviewed (3 files)
  • apps/web/src/routers/admin/blacklist-domains-router.ts
  • apps/web/src/app/admin/components/BlacklistedDomains.tsx
  • apps/web/src/routers/admin/blacklist-domains-router.test.ts

Reviewed by claude-sonnet-4.6 · 556,947 tokens

Review guidance: REVIEW.md from base branch main

@markijbema markijbema merged commit 5b41025 into main May 28, 2026
17 checks passed
@markijbema markijbema deleted the mark/bookmark-blacklisted-domain-tabs branch May 28, 2026 14:16
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