Skip to content

feat(web): simplify project modal, sidebar plus button, danger zone, curl webhooks#908

Merged
aaight merged 1 commit intodevfrom
feature/ui-updates-69b828173185a27f765e7e1b
Mar 16, 2026
Merged

feat(web): simplify project modal, sidebar plus button, danger zone, curl webhooks#908
aaight merged 1 commit intodevfrom
feature/ui-updates-69b828173185a27f765e7e1b

Conversation

@aaight
Copy link
Copy Markdown
Collaborator

@aaight aaight commented Mar 16, 2026

Summary

  • Simplify New Project modal — Keep only Name field; slug auto-generated from name, repo/branch are SCM integration concerns
  • Replace All Projects sidebar link with + button — Added + in PROJECTS header; project creation now accessible from sidebar
  • Add Danger Zone to General page — Delete project with confirmation dialog, navigates home on success
  • Fix Progress Interval display — Show as integer (5) not decimal (5.0)
  • Remove default model text from Progress Model description — Placeholder already shows default
  • Replace admin credentials with copyable curl instructions — PM webhook (Trello/JIRA) and GitHub webhook sections now show provider-specific curl commands instead of one-time credential inputs
  • Remove PM wizard SaveStep summary block — Removed the redundant Provider/Board/Lists summary and trigger config note; keep only the save button

Card: https://trello.com/c/69b828173185a27f765e7e1b

Changed Files

  • web/src/components/projects/project-form-dialog.tsx — Name-only form, auto-slugify
  • web/src/components/layout/sidebar.tsx+ button replaces All Projects link
  • web/src/components/projects/project-general-form.tsx — Danger zone, integer interval, simplified description
  • web/src/components/projects/pm-wizard-common-steps.tsx — Curl panel in WebhookStep, simplified SaveStep
  • web/src/components/projects/pm-wizard-hooks.ts — Remove oneTime* state and tokens
  • web/src/components/projects/integration-form.tsx — Curl panel for GitHub webhook section

Test plan

  • All 5415 unit tests pass
  • TypeScript typecheck passes (backend + frontend)
  • Biome lint passes with no errors
  • Manual: verify new project modal only shows Name field
  • Manual: verify + button in sidebar opens project creation dialog
  • Manual: verify Danger Zone delete on General page navigates to home
  • Manual: verify progress interval shows as 5 not 5.0
  • Manual: verify curl commands show in PM and GitHub webhook sections

🤖 Generated with Claude Code

🕵️ claude-code · claude-sonnet-4-6 · run details

Copy link
Copy Markdown
Collaborator

@nhopeatall nhopeatall left a comment

Choose a reason for hiding this comment

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

Summary

LGTM — Clean UI simplification with consistent patterns across all changes. All CI checks pass. The changes correctly remove the one-time admin credential UI in favor of copyable curl instructions, simplify the project creation form, add a Danger Zone for project deletion, and fix the progress interval display.

Should Fix (non-blocking)

  • web/src/components/projects/integration-form.tsx:112 & pm-wizard-common-steps.tsx:39 — Duplicated CopyButton component: The identical CopyButton component is defined in both files. Consider extracting to a shared component (e.g., web/src/components/ui/copy-button.tsx) to avoid maintaining the same code in two places.

  • web/src/components/projects/project-form-dialog.tsx:40 — Empty slug edge case: If a user enters a name consisting only of special characters (e.g., "!!!"), slugify produces an empty string. The backend's min(1) validation catches this, but the resulting Zod error message will be opaque. A client-side guard (e.g., disabling submit when slugify(name) is empty) would improve UX.

🕵️ claude-code · claude-opus-4-6 · run details

@aaight aaight merged commit 7b3df55 into dev Mar 16, 2026
6 checks passed
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