Skip to content

fix(dashboard): mount ProjectIdentity in BuildSettings to allow project name and description editing - #313

Open
farrasrayhand wants to merge 17 commits into
oblien:mainfrom
farrasrayhand:fix/dashboard-project-rename-identity
Open

fix(dashboard): mount ProjectIdentity in BuildSettings to allow project name and description editing#313
farrasrayhand wants to merge 17 commits into
oblien:mainfrom
farrasrayhand:fix/dashboard-project-rename-identity

Conversation

@farrasrayhand

@farrasrayhand farrasrayhand commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Mounts the existing <ProjectIdentity /> card on the project settings page (BuildSettings.tsx) so users can view and edit their Project Name and Description in the Dashboard UI.

Motivation

Users were unable to find where to rename projects in the dashboard (Issue #278). The backend API (PATCH /projects/:id) and the frontend <ProjectIdentity /> component (which handles calling projectsApi.update(id, { name, description })) were already fully implemented, but <ProjectIdentity /> was unmounted.

Related issue

#278

Changes

  • apps/dashboard/src/app/(dashboard)/projects/[id]/components/BuildSettings.tsx:
    • Imported and mounted <ProjectIdentity /> at the top of the project settings / runtime configuration surface.

Verification

Manual verification in Dashboard UI:

  1. Navigate to Project → Settings / Configuration.
  2. The "Project Identity" card appears at the top displaying Project Name & Description.
  3. Click "Edit", change Project Name, and click "Save". Toast notification confirms success and Project Name updates in UI.

Checklist

  • One change per PR — one bug, or one agreed feature, with nothing unrelated bundled in
  • The diff is scoped — no reformatting or lint fixes on lines I wasn't otherwise changing
  • A test fails without this change and passes with it (or I explained above why there isn't one)
  • bun run test, bun run --cwd <workspace> lint, and bun format all pass locally
  • I understand every line of this diff and can explain it in review

farrasrayhand and others added 16 commits July 25, 2026 08:11
…r compat)

GNU tar does not support --no-absolute-names; it strips absolute paths by
default. The pre-extraction assertTarEntriesSafe already validates all
entry paths for traversal attacks.
…blicWebhookUrl

- Replace assertPublicWebhookUrl call with assertPublicUrlLiteral from ssrf-guard
- Delete redundant webhook-url.ts and its test
- Add 22 ssrf-guard tests covering isPrivateIp, isBlockedHostname, assertPublicHostLiteral, assertPublicUrlLiteral
@farrasrayhand
farrasrayhand force-pushed the fix/dashboard-project-rename-identity branch 2 times, most recently from b286a55 to 41f6858 Compare July 30, 2026 03:21
@farrasrayhand
farrasrayhand force-pushed the fix/dashboard-project-rename-identity branch from 41f6858 to 3b5a3a4 Compare July 30, 2026 03:26
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.

1 participant