Sss 4 opt stabilization to release #20

coderabbitai · 2025-04-08T02:29:26Z

💡 Verification agent

🧩 Analysis chain

Confirm no leftover references to async usage.

Previously, you had DashboardPage as an async function. Changing it to a standard function is fine, but please ensure no upstream code still relies on an async export.

🏁 Script executed:

#!/bin/bash # Description: Search potential references to the old async function name. fd -e tsx -e ts --exec rg "DashboardPage" {}

Length of output: 94

Action Required: Remove or Update Leftover Async Declaration

It appears that a leftover async export for DashboardPage is still present in the codebase:

Detected Code:
export default async function DashboardPage() {

Impact:
This suggests that somewhere in the repository, the old async function is still defined. Please verify if any upstream code depends on the async behavior of DashboardPage. If not needed, remove or refactor this leftover export to align with the new synchronous implementation to avoid any unintended side effects.

-Original file line number
+Diff line change
@@ Expand Up / @@ -63,7 +63,7 @@ knowledge_base: @@
       issues:
         scope: auto
       jira:
-        project_keys: []
+        project_keys: [SSS]
       linear:
         team_keys: []
       pull_requests:
@@ Expand Down @@

-Original file line number
+Diff line change
@@ -1,38 +1,12 @@
-    import { Link } from '@opt/navigations';
-    import { ActionButton } from '@repo/ui';
+    import { DashboardContent } from '@opt/components/dashboard';
+    import { TripsProvider } from '@opt/components/providers/TripsProvider';
-    export default async function DashboardPage() {
+    export default function Dashboard() {
       return (
-        <div>
-          <header>
-            <h2>Your trips</h2>
-            <ActionButton>new Trip</ActionButton>
-          </header>
-          <article>
-            <h3>Current trip</h3>
-            <p>You can see your past trips here.</p>
-            <p>
-              To create a new trip, click on <strong>New Trip</strong>.
-            </p>
-            <p>Click on a trip to view its details.</p>
-          </article>
-          <article>
-            <h3>Coming soon...</h3>
-            <p>You can see your past trips here.</p>
-            <p>
-              To create a new trip, click on <strong>New Trip</strong>.
-            </p>
-            <p>Click on a trip to view its details.</p>
-          </article>
-          <article>
-            <h3>Trips history</h3>
-            <p>You can see your past trips here.</p>
-            <p>
-              To create a new trip, click on <strong>New Trip</strong>.
-            </p>
-            <p>Click on a trip to view its details.</p>
-          </article>
-          <Link href={'/trips'}>See all trips</Link>
-        </div>
+        <TripsProvider>
+          <div className="dashboard">
+            <DashboardContent />
+          </div>
+        </TripsProvider>
       );
     }

-Original file line number
+Diff line change
@@ -1,24 +1,36 @@
     import '@opt/styles/main.scss';
-    import { getMessages, unstable_setRequestLocale } from 'next-intl/server';
+    import { getMessages, setRequestLocale } from 'next-intl/server';
+    import { auth } from '@opt/auth';
     import { AppWrapper, SWRProvider } from '@opt/components/UI';
+    import { routing } from '@opt/i18n/routing';
     import { Navbar } from '@repo/ui';
-    import { PageProps } from '@repo/ui/types';
+    import { OPT_MENU_ITEMS } from '@repo/ui/constants';
     import { metadata } from '../metadata';
-    const languages = ['en', 'es'];
     export async function generateStaticParams() {
-      return languages.map((locale) => ({ locale }));
+      return routing.locales.map((locale) => ({ locale }));
     }
     export default async function RootLayout({
       children,
-      params: { locale },
-    }: PageProps) {
-      unstable_setRequestLocale(locale);
+      params,
+    }: {
+      children: React.ReactNode;
+      params: { locale: 'es' | 'en' };
+    }) {
+      const { locale } = await params;
+      setRequestLocale(locale);
+      const session = await auth();
       const messages = await getMessages();
+      const authenticated = !!session;
+      const menuItems = OPT_MENU_ITEMS;
       return (
         <html lang="en" suppressHydrationWarning>
           <body>
@@ Expand All / @@ -27,9 +39,7 @@ export default async function RootLayout({ @@
                 <main className="layout">
                   <Navbar
                     title={'One Plan Trip'}
-                    menuItems={[]}
-                    authenticated
-                    locale={locale}
+                    {...{ authenticated, menuItems, locale }}
                   />
                   <div className="layout__content">
                     <section className="layout__section">{children}</section>
@@ Expand Down @@

-Original file line number
+Diff line change
@@ -1,7 +1,7 @@
     'use client';
     import { ArrowLeft } from 'lucide-react';
-    import { useRouter } from '@opt/navigations';
+    import { useRouter } from '@opt/i18n/routing';
     import { ActionButton } from '@repo/ui';
     export default function NotFound() {
@@ Expand Down @@

-Original file line number
+Diff line change
@@ -1,7 +1,24 @@
-    export default function Web() {
+    import { getTranslations } from 'next-intl/server';
+    import Image from 'next/image';
+    import { Link } from '@opt/i18n/routing';
+    import landing_img from '@repo/ui/assets/animated/globalization-a.svg';
+    export default async function HomePage() {
+      const t = await getTranslations('home');
       return (
-        <div>
-          <h1 className="heading-1">Este es One plan trip</h1>
+        <div className="home__container">
+          <div className="home__title">
+            <h1 className="heading--1">{t('title2')}</h1>
+            <h5 className="subtitle--1">{t('subtitle2')}</h5>
+            <Link href={`/dashboard`} className="home__button">
+              {t('tryNow')}
+            </Link>
+          </div>
+          <div className="home__title">
+            <Image src={landing_img} alt="traveling image" width={550} />
+          </div>
         </div>
       );
     }

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sss 4 opt stabilization to release #20

Uh oh!

Diff view

Diff view

There are no files selected for viewing

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot Apr 8, 2025

Uh oh!

Uh oh!

-Original file line number
+Diff line change
@@ -1,5 +1,8 @@
     import { TripDetails } from '@opt/components/trips';
-    export default function Trip({ params }: { params: { id: string } }) {
-      return <TripDetails id={params.id} />;
+    type IdParams = Promise<{ id: string }>;
+    export default async function Trip({ params }: { params: IdParams }) {
+      const { id } = await params;
+      return <TripDetails id={id} />;
     }

-Original file line number
+Diff line change
@@ -1,23 +1,10 @@
-    import { redirect } from 'next/navigation';
-    import { TripsContainer } from '@opt/components/trips';
-    import { ActionButton } from '@repo/ui';
+    import { TripsProvider } from '@opt/components/providers/TripsProvider';
+    import { TripsContent } from '@opt/components/trips';
     export default async function Trips() {
       return (
-        <div>
-          <header>
-            <h2>Trips</h2>
-            <form
-              action={async () => {
-                'use server';
-                return redirect('/trips/new');
-              }}
-            >
-              <ActionButton variant="primary">Create new Trip</ActionButton>
-            </form>
-          </header>
-          <TripsContainer />
-        </div>
+        <TripsProvider>
+          <TripsContent />
+        </TripsProvider>
       );
     }

-Original file line number
+Diff line change
@@ Expand Up / @@ -3,8 +3,8 @@ import { signOut, useSession } from 'next-auth/react'; @@
     import { toast } from 'react-hot-toast';
     import { Trip } from '@opt/core/interfaces';
+    import { useRouter } from '@opt/i18n/routing';
     import { TripActions } from '@opt/integration/actions/';
-    import { useRouter } from '@opt/navigations';
     import { useTripStore } from '@opt/store';
     import { ActionButton } from '@repo/ui/';
@@ Expand Down @@

-Original file line number
+Diff line change
@@ -0,0 +1,20 @@
+    'use client';
+    import { useTripsContext } from '../providers/TripsProvider';
+    import { TripSearch, TripSections, TripStats } from '../trips';
+    export const DashboardContent = () => {
+      const { trips, isLoading, isError } = useTripsContext();
+      if (isLoading) return <div>Loading...</div>;
+      if (isError) return <div>Error loading trips.</div>;
+      return (
+        <>
+          <div className="dashboard__search">
+            <TripSearch />
+          </div>
+          <TripStats trips={trips} />
+          <TripSections trips={trips} />
+        </>
+      );
+    };

Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		export { DashboardContent } from './DashboardContent';

Sss 4 opt stabilization to release #20

Are you sure you want to change the base?

Uh oh!

Sss 4 opt stabilization to release #20

Uh oh!

Uh oh!

Diff view

Diff view

There are no files selected for viewing

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot Apr 8, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!