feat(listings): publish/unpublish CTA + real reviews + host data + dual-track docs#10
Open
abdout wants to merge 9 commits into
Open
feat(listings): publish/unpublish CTA + real reviews + host data + dual-track docs#10abdout wants to merge 9 commits into
abdout wants to merge 9 commits into
Conversation
…d-start The pg adapter holds long-lived TCP connections that Neon drops when its serverless compute scales to zero, causing the next query to fail with "Server has closed the connection". The neon serverless adapter speaks HTTPS+WS and wakes the compute on demand. Detect Neon URLs and switch automatically when DATABASE_URL_ADAPTER is unset. Pairs with a backfill of the missing Location.createdAt/updatedAt columns applied via Neon MCP — schema.prisma listed them but production never received the ALTER TABLE, so every Listing.findMany() failed once the nested location include resolved. Refs #4
…apes, restructure meta
- Replace pathname.startsWith("/ar") heuristic with the existing
reportIssue.* dictionary slice via useDictionary(). A single locale
source keeps the widget consistent with every other localized surface.
- Use literal Arabic glyphs in fallback handling instead of \u escape
sequences (carry-over from the previous edit; Arabic is now sourced
from the dictionary anyway).
- Pull browser/viewport/direction into a nested `meta` object on the
server action's input shape so the body fields are always emitted in
the same canonical order — needed by the kun report-agent parser.
- Self-bootstrap the `report` GitHub label on 422 with color #d93f0b so
the action works in any repo without manual seeding.
Refs #5
…link Mounts <ReportIssue variant="icon"/> as a fixed-position floating button in the locale-root layout so every one of the 117 locale-prefixed routes exposes the report widget. Previously only two footer components carried it, leaving auth, hosting, dashboard, transport, and admin surfaces with no path to file a bug. Adds the explicit Next 16 viewport export (width/initialScale/themeColor) that mobile Safari needs to render at the correct DPR and to honour prefers-color-scheme for the address bar. Moves the bilingual skip-link literal at layout.tsx:74 to common.skipToContent in both dictionaries — the only remaining inline ternary in the locale-root layout. Refs #5
…consolidated proxy - Flip `i18n.defaultLocale` from `en` to `ar` per the global rule. The proxy already cookie-pins NEXT_LOCALE, so visitors with a prior English preference are not redirected; only fresh, headerless visits land on /ar/*. - Make `useDictionary()` lenient — return the bundled English JSON as a static fallback when no DictionaryProvider is mounted instead of throwing, so test fixtures, isolated demos, and the root error page render strings. - Drop the duplicated `src/components/internationalization/middleware.ts`. Its `localizationMiddleware` export was never imported (proxy.ts is the live entry point in Next 16) and the parallel logic was a footgun. - Update `[lang]/layout.tsx` so the default-lang and default-config fallbacks reference `i18n.defaultLocale` instead of a hardcoded "en", and the hreflang `x-default` mirrors the same. Metadata API auto-emits the alternate hreflang links. - Add `src/lib/i18n/date-locale.ts` (`dateLocaleFor` + `intlLocaleFor`) and `src/components/ui/direction-aware-icon.tsx` so future code has one place to look for "date-fns locale per app locale" and "icon flip on RTL". Refs #7
…n, transport metadata, and skipToContent Adds the keys the upcoming refactor needs so per-page generateMetadata, filter panels, paginators, and the skip-link can pull from the dictionary instead of inline `lang === 'ar' ? ... : ...` ternaries: - `common.skipToContent` (replaces the bilingual literal at layout.tsx:74) - `pageMetadata.<page>` objects with `title` + `description` + (optional) `subtitle` for help / host / hosting / hostingListings / transportHost / transportOffices / transportCheckout / landing / login / join / reset / managersProperties / tenantsFavorites / tenantsResidences - `rental.property.filters` — `title`, `clearAll`, `any`, `priceRange` - `rental.property.pagination` — `previous`, `next`, `pageOf` - `transport.search.metadataTitle` + `metadataDescription` + `pagination` - `transport.routes.hoursFormat` + `currency` - `transport.metadataTitle` + `metadataDescription` Both en.json and ar.json updated in parity. Refs #7
…String calls
Every visible string and date now flows through the dictionary or through
the locale-aware formatter helpers. Concrete swaps:
- generateMetadata across 17 pages now pulls from `dictionary.pageMetadata.*`
instead of `lang === 'ar' ? "..." : "..."` ternaries.
- `transport/page.tsx`, `transport/search/page.tsx`, `listings/page.tsx`,
`transport/booking/checkout/content.tsx`, etc. drop their per-key
ternary fallbacks now that the dictionary has the keys upstream.
- Date display sites — bookings, transport-host, dashboard, tenants,
managers, admin tables, ApplicationCard — call `formatDate(date, lang)`
from `src/lib/i18n/formatters.ts` instead of `new Date(x).toLocaleDateString()`.
Admin tables receive `lang` as a prop from their server-page caller.
- Locale narrowing in client components (`useParams<{lang}>().lang === 'ar' ? 'ar' : 'en'`)
is replaced by `useLocale()` which returns the typed `Locale`.
- Data-shape ternaries (category.labelAr/label, tw.rangeAr/En, DEMO_DATA.ar/en)
are restructured to be locale-keyed objects so call sites do
`obj[lang]` instead of branching.
- The decorative `data-day` attribute in `ui/calendar.tsx` switches to a
stable ISO date so the value is locale-independent.
Closes the i18n anti-pattern audit's `inline lang ternaries` and
`raw toLocaleDateString calls` budgets at zero.
Refs #7
…t + lefthook gate - `ui/carousel.tsx`, `ui/calendar.tsx`, `listings/pagination.tsx`, `[lang]/hosting/calendar/page.tsx`, and the manager-applications back-button each pick up `rtl:rotate-180` so their chevrons and arrows point the right way under RTL. Decorative arrows whose meaning is rotation-independent (e.g. the ticket-page absolute-positioned arrow) are intentionally left as-is. - `HeartButton` and the `<ReportIssue variant="icon" />` trigger now wrap their svg/lucide icon in a 44x44 hit area (`h-11 w-11 inline-flex`), matching iOS HIG and WCAG 2.5.5 (Target Size, AA). - New `scripts/i18n-anti-pattern-check.sh` greps for inline lang ternaries, raw toLocaleDateString, raw toLocaleString, and likely-hardcoded English JSX. Each metric has an env-overridable budget; the budget can only shrink. Wired as `pnpm i18n:check` and added to lefthook pre-commit. - `tests/e2e/seo.spec.ts` extended with three new assertions: html lang+dir parity per locale (en/ltr, ar/rtl) and presence of the report-issue mount on every locale-prefixed surface. Refs #7
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Final pieces of Phase C1: - Mobile listing detail now passes the latest 8 reviews + average rating + total count into <MobileReviews/>. The component drops its sample-data fallback and renders an empty state when there are no reviews. Review date is locale-aware via formatDate. - /host/content.tsx no-op `handleCreateFromExisting` TODO is replaced with a route to the host overview (where the existing listings are pickable). Clone-from-existing is filed as a follow-up. - New `docs/booking-vs-application.md` documents the dual-track rental flow (short-term Booking vs. long-term Application -> Lease) and the rule that they must not be mixed in a single UI flow. Refs #9
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This was referenced Apr 28, 2026
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
1 participant
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.
Summary
Phase C1 of the production-readiness push (plan:
kun/.claude/plans/resume-the-development-in-abstract-rossum.md). Built on top of #6 + #8.<PublishToggleButton>client island wraps the existingpublishListing/unpublishListingserver actions inuseTransition. Renders per row in/hosting/listingsand inherits the action's server-side validation (required fields, location, ≥1 photo) — surfaces its error message via toast.Published,In progress,Action required) now resolve throughdictionary.hostingListings.status.*instead of hardcoded English strings.<Reviews>becomes an async server component that fetches viagetListingReviews(listingId, { take: 6 })and renders an empty statedictionary.rental.reviewsList.emptywhen there are zero.<MobileReviews>likewise takes real review props and drops its 4-row sample fallback. Review dates flow throughformatDate(date, lang). Server-rendered<Review>and<MeetHost>slots are threaded intoListingDetailsClientso the client island doesn't have to refetch on hydrate.<HostedBy>,<MeetHost>,<MobileMeetHost>accept the host user (username/email/image) + review aggregates as props. No more "Hosted by Faisal" with a stranger's stock photo. Falls back to email-prefix when username is null and to a translated "Host" label when both are.docs/booking-vs-application.mddescribes the two paths (short-term Booking vs. long-term Application → Lease) and the rule that they don't interoperate. Saves future contributors from refactor mistakes.host/content.tsxTODO:handleCreateFromExistingno longer no-ops — it routes to the host overview where existing listings are pickable. Clone-from-existing is filed as a follow-up.Verification
pnpm typecheckclean.pnpm i18n:checkPASS — 0 inline ternaries, 0 rawtoLocaleDateString./listings.listing.host.username || email-prefix— never "Faisal".Out of scope
Listing.rentalTypefield migration (Phase D — currently inferred).docs/booking-vs-application.md; both paths stay).Refs #9