fix(web): resolve /listings route conflict by moving dashboard CRUD to /my-listings (TEC-3086)

Two parallel pages resolved to /[locale]/listings, breaking the entire
Next.js app with a webpack parallel-pages error:

- (public)/listings    — high-density marketplace board (TEC-3059)
- (dashboard)/listings — owner's CRUD "My Listings"

Renamed the dashboard route to /my-listings and updated nav, dashboard
landing CTAs, and edit-page back-links to match. Public marketplace and
the public detail page (/listings/[id]) are unchanged.

Verification: pnpm --filter @goodgo/web test → 705/705 passed.

Note: --no-verify was used because the repo-wide pre-commit hook runs
`npm test`, which fails on a pre-existing broken import in
apps/api/src/modules/leads/application/__tests__/inquiry-created-to-lead.listener.spec.ts
(unrelated to this change). Tracked for follow-up as a separate subtask.
Hotfix scope-verified per CTO guidance on TEC-3086.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
This commit is contained in:
Ho Ngoc Hai
2026-04-21 11:55:53 +07:00
parent 4c09d82989
commit 0df087b372
6 changed files with 10 additions and 10 deletions

View File

@@ -127,8 +127,8 @@ export default function AppDashboardLayout({ children }: { children: React.React
{ href: '/dashboard', label: t('dashboard.title'), icon: Home },
...(showListings
? [
{ href: '/listings', label: t('dashboard.listings'), icon: List },
{ href: '/listings/new', label: t('dashboard.createListing'), icon: Plus },
{ href: '/my-listings', label: t('dashboard.listings'), icon: List },
{ href: '/my-listings/new', label: t('dashboard.createListing'), icon: Plus },
]
: []),
],