Publish Ukiyo-e portfolio dashboard - #6
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR transitions the web app from the retired “育爱成长 / Nurture Bloom” shell to a Ukiyo-e themed portfolio dashboard, updating routing and identity to publish a project showcase with evidence links and adding CI verification + publishing metadata.
Changes:
- Replaces the previous nurture feature routes/content with new
/dashboardand/evidence/$projectIdroutes and portfolio UI. - Updates
/to redirect to/dashboard, and refreshes document metadata + product identity text. - Adds Baby2B publish manifest and a reusable GitHub Actions verification workflow.
Reviewed changes
Copilot reviewed 25 out of 27 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/web/apps/web/src/routes/moments.tsx | Removes the /moments route. |
| apps/web/apps/web/src/routes/me.tsx | Removes the /me route. |
| apps/web/apps/web/src/routes/guide.tsx | Removes the /guide route. |
| apps/web/apps/web/src/routes/growth.tsx | Removes the /growth route. |
| apps/web/apps/web/src/routes/index.tsx | Redirects / from /growth to /dashboard. |
| apps/web/apps/web/src/routes/dashboard.tsx | Adds the new /dashboard route. |
| apps/web/apps/web/src/routes/evidence.$projectId.tsx | Adds the new /evidence/$projectId route. |
| apps/web/apps/web/src/routes/__root.tsx | Updates site title/description metadata to the portfolio identity. |
| apps/web/apps/web/src/features/portfolio/dashboard-content.tsx | Implements the Ukiyo-e portfolio dashboard UI, project cards, and navigation. |
| apps/web/apps/web/src/features/portfolio/evidence-content.tsx | Implements per-project evidence/details rendering (including not-found state). |
| apps/web/apps/web/src/data/portfolio-projects.ts | Adds the portfolio project dataset and index for evidence lookup. |
| apps/web/apps/web/src/components/layout/top-bar.tsx | Updates top bar branding/identity for the showcase. |
| apps/web/apps/web/src/components/layout/bottom-nav.tsx | Collapses bottom nav to a single dashboard entry. |
| apps/web/apps/web/src/components/layout/app-shell.tsx | Simplifies the shell (removes fixed top/bottom layout wrappers). |
| apps/web/apps/web/src/components/app-identity.tsx | Updates visible product identity string. |
| apps/web/apps/web/src/components/tests/app-identity.test.tsx | Updates identity test expectation to “Showcase Dashboard”. |
| apps/web/apps/web/src/features/nurture/types.ts | Removes nurture domain types. |
| apps/web/apps/web/src/features/nurture/profile-content.tsx | Removes nurture profile UI. |
| apps/web/apps/web/src/features/nurture/moments-content.tsx | Removes nurture moments UI. |
| apps/web/apps/web/src/features/nurture/guide-content.tsx | Removes nurture guide UI. |
| apps/web/apps/web/src/features/nurture/growth-content.tsx | Removes nurture growth UI. |
| apps/web/apps/web/src/features/nurture/data.ts | Removes nurture seed data. |
| apps/web/apps/web/src/features/nurture/tests/local-interactions.test.tsx | Removes nurture interaction tests tied to removed features. |
| .github/workflows/verify-baby2b-project.yml | Adds verification workflow using a reusable workflow with build/test/typecheck steps. |
| .github/baby2b-publish.yml | Adds publish manifest describing build output and production URLs. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+92
to
+98
| <button | ||
| aria-label="打开菜单" | ||
| className="grid size-11 place-items-center text-[#071d34] md:hidden" | ||
| type="button" | ||
| > | ||
| <Menu aria-hidden="true" size={23} /> | ||
| </button> |
Comment on lines
+191
to
+196
| <a | ||
| className="group block min-w-0 overflow-hidden border border-[#c7ced8] bg-white shadow-sm transition hover:-translate-y-0.5 hover:border-[#bf1737]/50 hover:shadow-md focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-4 focus-visible:outline-[#bf1737]" | ||
| href={project.evidenceUrl} | ||
| key={project.id} | ||
| rel="noreferrer" | ||
| > |
|
|
||
| jobs: | ||
| verify: | ||
| uses: Tiancheng-Xu/.github/.github/workflows/verify-project.yml@main |
Comment on lines
+13
to
+23
| export function EvidenceContent({ projectId }: { projectId: string }) { | ||
| const project = PROJECTS_INDEX[projectId]; | ||
|
|
||
| if (!project) { | ||
| return ( | ||
| <div className="space-y-4"> | ||
| <header> | ||
| <h1 className="font-bold text-2xl">项目不存在</h1> | ||
| <p className="mt-1 text-muted-foreground text-sm"> | ||
| 该卡片已失效或未在看板中配置，建议返回看板后重试。 | ||
| </p> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
/redirect to/dashboardNavigation contract
https://baby2b.online/https://baby2b.online/dashboardhttps://evidence.baby2b.online/Validation