PetschStudios/KanbanGit
3
0
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases 24 Wiki Activity

KAN-45: Rebuild project notification settings #140

Merged
Twyco merged 1 commits from feature/KAN-45-project-notification-settings into main 2026-09-16 07:32:22 +00:00
Conversation 0 Commits 1 Files Changed 11
+1752 -1190
Twyco commented 2026-09-15 20:44:42 +00:00
Owner
Copy Link
Copy Source

Change

KAN-45 rebuilds project notification settings in the shared project settings layout. Notification actions now have a compact overview and a shared create/edit dialog with existing triggers, channels, Discord templates, mentions and filters. Deletion requires confirmation, and validation, server and network failures retain the form for retry.

Secret webhook URLs stay masked during entry, are never populated from stored values, and are cleared on success or discard. Existing notification behavior and template inheritance are preserved. All new interface text uses translation keys.

Validation

  • 13 targeted PHPUnit tests passed (139 assertions), including array filter payloads, clearing mentions and unchanged configuration after invalid updates.
  • Browser tests passed for create/edit/delete payloads, channel changes, filter editing, secret handling, validation, 403/500/network retries, dirty forms and keyboard focus.
  • Responsive checks passed at 320, 390, 768 and 1440 pixels in light and dark modes.
  • Production build, repository ESLint, final TypeScript check, Pint and diff checks passed.

Base: main, pulled with git pull --ff-only origin main before branch creation.

## Change KAN-45 rebuilds project notification settings in the shared project settings layout. Notification actions now have a compact overview and a shared create/edit dialog with existing triggers, channels, Discord templates, mentions and filters. Deletion requires confirmation, and validation, server and network failures retain the form for retry. Secret webhook URLs stay masked during entry, are never populated from stored values, and are cleared on success or discard. Existing notification behavior and template inheritance are preserved. All new interface text uses translation keys. ## Validation - 13 targeted PHPUnit tests passed (139 assertions), including array filter payloads, clearing mentions and unchanged configuration after invalid updates. - Browser tests passed for create/edit/delete payloads, channel changes, filter editing, secret handling, validation, 403/500/network retries, dirty forms and keyboard focus. - Responsive checks passed at 320, 390, 768 and 1440 pixels in light and dark modes. - Production build, repository ESLint, final TypeScript check, Pint and diff checks passed. Base: `main`, pulled with `git pull --ff-only origin main` before branch creation.
Twyco added 1 commit 2026-09-16 07:32:15 +00:00
feat(KAN-45): rebuild project notification settings
CI/CD / preconditions (pull_request) Canceled after 0s
Details
CI/CD / test_and_check_app (pull_request) Canceled after 0s
Details
CI/CD / Publish immutable image (pull_request) Canceled after 0s
Details
538b17b9b8
Twyco force-pushed feature/KAN-45-project-notification-settings from 3f795ba32e to 538b17b9b8 2026-09-16 07:32:15 +00:00 Compare
Twyco merged commit 0a974cfd49 into main 2026-09-16 07:32:22 +00:00
Twyco deleted branch feature/KAN-45-project-notification-settings 2026-09-16 07:32:22 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Eric Twyco flux-image-automation (Flux Image Automation)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: PetschStudios/KanbanGit#140