Dashboard page reworks (#677)

* ensure application rating uniqueness per rater

* hardcode share link app url

* logout endpoint

* fix logout endpoint

* add "logout" button to dashboard sidebar

* dev mode for inviting members

* improved invite page

* fix usage of `getCurrentUser`

Author: KavikaPalletenne

backend/server/src/handler/auth.rs

@@ -153,6 +153,32 @@ pub async fn google_callback(
     Ok((jar.add(cookie), Redirect::to(redirect_url.as_str())))
 }
 
+pub async fn logout(
+    State(state): State<AppState>,
+    jar: CookieJar
+) -> Result<impl IntoResponse, ChaosError> {
+    let domain = if state.is_dev_env {
+        "localhost"
+    } else {
+        "devsoc.app"
+    };
+
+    let empty_cookie= Cookie::build(("auth_token", ""))
+    .http_only(true) // Prevent JavaScript access
+    .expires(Expiration::DateTime(OffsetDateTime::now_utc() + time::Duration::days(5))) // Set an expiration time of 5 days, TODO: read from env?
+    .secure(!state.is_dev_env)     // Send only over HTTPS, comment out for testing
+    .domain(domain)
+    .path("/");
+
+    let redirect = if state.is_dev_env {
+        "http://localhost:3000"
+    } else {
+        "https://chaos.devsoc.app"
+    };
+
+    Ok((jar.remove(empty_cookie), Redirect::to(redirect)))
+}
+
 pub struct DevLoginHandler;
 impl DevLoginHandler {
 

backend/server/src/handler/organisation.rs

@@ -381,6 +381,7 @@ impl OrganisationHandler {
             admin.user_id,
             request_body.email,
             state.email_credentials.clone(),
+            state.is_dev_env,
             &mut state.snowflake_generator,
             &mut transaction.tx,
         )

backend/server/src/models/app.rs

@@ -1,6 +1,6 @@
 use crate::handler::answer::AnswerHandler;
 use crate::handler::application::ApplicationHandler;
-use crate::handler::auth::{google_callback, google_auth_init, DevLoginHandler};
+use crate::handler::auth::{DevLoginHandler, google_auth_init, google_callback, logout};
 use crate::handler::campaign::CampaignHandler;
 use crate::handler::email_template::EmailTemplateHandler;
 use crate::handler::offer::OfferHandler;
@@ -201,6 +201,7 @@ pub async fn app() -> Result<(Router, AppState), ChaosError> {
     let router = Router::new()
         .route("/", get(|| async { "Join DevSoc! https://devsoc.app/" }))
         .route("/auth/google", get(google_auth_init))
+        .route("/auth/logout", get(logout))
         .route("/api/auth/callback/google", get(google_callback))
         .route("/api/v1/dev/super_admin_login", get(DevLoginHandler::dev_super_admin_login))
         .route("/api/v1/dev/org_admin_login", get(DevLoginHandler::dev_org_admin_login))

backend/server/src/models/application.rs

@@ -668,7 +668,7 @@ impl Application {
                     
                     coalesce(
                         to_jsonb(
-                            array_agg(
+                            array_agg(DISTINCT
                                 jsonb_build_object(
                                     'id', ar.id,
                                     'rater_id', reviewer.id,
@@ -690,7 +690,7 @@ impl Application {
                                         WHERE crc.campaign_id = a.campaign_id
                                     ),
                                     'updated_at', ar.updated_at
-                                ) ORDER BY ar.updated_at DESC
+                                )
                             ) FILTER (WHERE ar.id IS NOT NULL)
                         ),
                         '[]'::jsonb

backend/server/src/models/organisation.rs

@@ -745,6 +745,7 @@ impl Organisation {
         inviting_user_id: i64,
         email: String,
         email_credentials: EmailCredentials,
+        is_dev_env: bool,
         snowflake_generator: &mut SnowflakeIdGenerator,
         transaction: &mut Transaction<'_, Postgres>,
     ) -> Result<String, ChaosError> {
@@ -830,14 +831,18 @@ impl Organisation {
         .execute(transaction.deref_mut())
         .await?;
 
-        ChaosEmail::send_message(
-            None,
-            email,
-            "You have been invited to join an organisation on Chaos".to_string(),
-            format!("You have been invited to join an organisation on Chaos. Please use the following link to accept the invite: https://chaos.devsoc.app/dashboard/invite/{code}").to_string(),
-            email_credentials
-        )
-        .await?;
+        if is_dev_env {
+            println!("Invite code for {email}: {code}")
+        } else {
+            ChaosEmail::send_message(
+                None,
+                email,
+                "You have been invited to join an organisation on Chaos".to_string(),
+                format!("You have been invited to join an organisation on Chaos. Please use the following link to accept the invite: https://chaos.devsoc.app/dashboard/invite/{code}").to_string(),
+                email_credentials
+            )
+            .await?;
+        }
 
         return Ok(code);
 }

backend/server/src/models/rating.rs

@@ -317,6 +317,8 @@ impl Rating {
             "
                 INSERT INTO application_ratings (id, application_id, rater_id, comment)
                     VALUES ($1, $2, $3, $4)
+                ON CONFLICT (application_id, rater_id)
+                DO UPDATE SET comment = $4, updated_at = NOW()
             ",
             rating_id,
             application_id,

frontend-nextjs/src/app/[lang]/dashboard/invite/[code]/invite-client.tsx

@@ -7,13 +7,16 @@ import Link from "next/link";
 import { useEffect, useState } from "react";
 import { useRouter } from "next/navigation";
 import { useQuery } from "@tanstack/react-query";
+import { User } from "@/models/user";
+import { LogOut } from "lucide-react";
 
 type Props = {
   code: string;
+  currentUser?: User;
   dict: any;
 };
 
-export default function InviteClient({ code, dict }: Props) {
+export default function InviteClient({ code, currentUser, dict }: Props) {
   const router = useRouter();
 
   const [status, setStatus] = useState<"idle" | "loading" | "success" | "error">("idle");
@@ -46,18 +49,17 @@ export default function InviteClient({ code, dict }: Props) {
   const inviteInvalid = !invite || invite.expired || invite.used || status === "success";
 
   return (
-    <div className="max-w-xl mx-auto p-6 flex flex-col gap-4">
+    <div className="max-w-3xl mx-auto p-6 flex flex-col gap-4">
       <h1 className="text-2xl font-bold">{dict.dashboard.invite.title}</h1>
-      <p className="text-sm text-muted-foreground">
+      <p className="text-lg">
         {invite?.organisation_name}{" "}
-        {invite
-          ? dict.dashboard.invite.invited_by
-          : dict?.common?.loading ?? "Loading..."}
+        {dict.dashboard.invite.invited_by}
       </p>
       {/* Show the email the invite was sent to */}
       {invite && (
-        <p className="text-sm text-muted-foreground">
-          {dict.dashboard.invite.sent_to}: <span className="font-medium">{invite.email}</span>
+        <p className="text-lg">
+          {dict.dashboard.invite.sent_to}: <span className="font-medium">{invite.email}</span>.
+          {(!currentUser || currentUser.email !== invite.email) && dict.dashboard.invite.sent_to_note}
         </p>
       )}
       {/* Show the expired message if the invite has expired */}
@@ -74,25 +76,41 @@ export default function InviteClient({ code, dict }: Props) {
           {message}
         </p>
       )}
-      <div className="flex flex-col gap-2">
-        <Link href={`/login?to=/dashboard/invite/${code}`} className="w-full">
-          <Button variant="outline" className="w-full">
-            {dict.dashboard.invite.login_cta}
+      {
+        currentUser && currentUser.email === invite?.email && (
+          <Button onClick={handleAccept} disabled={status === "loading" || inviteInvalid} className="w-full">
+            {status === "loading" ? "Loading..." : dict.dashboard.invite.accept_cta}
           </Button>
-        </Link>
-        <Button
-          onClick={handleAccept}
-          disabled={status === "loading" || inviteInvalid}
-          className="w-full"
-        >
-          {status === "loading" ? "Loading..." : dict.dashboard.invite.accept_cta}
-        </Button>
-        {/* Show the wrong account message if the account is not invited */}
-        <p className="text-xs text-muted-foreground">
-          {dict.dashboard.invite.wrong_account}
-        </p>
-      </div>
-    </div>
+        )
+      }
+      {/* Show the wrong account message if the account is not invited */}
+      {
+        currentUser && currentUser.email !== invite?.email && (
+          <>
+          <p className="text-xl">
+            {dict.dashboard.invite.wrong_account}
+          </p>
+          <Link href={`${process.env.NEXT_PUBLIC_API_BASE_URL || "https://chaos-api.devsoc.app"}/auth/logout`} className="w-full">
+              <Button variant="outline" className="w-full">
+                <LogOut />
+                {dict.common.logout}
+              </Button>
+            </Link>
+          </>
+        )
+      }
+      {
+        !currentUser && (
+          <div className="flex flex-col gap-2">
+            <Link href={`/login?to=/dashboard/invite/${code}`} className="w-full">
+              <Button variant="outline" className="w-full">
+                {dict.dashboard.invite.login_cta}
+              </Button>
+            </Link>
+          </div>
+        )
+      }
+    </div >
   );
 }
 

frontend-nextjs/src/app/[lang]/dashboard/invite/[code]/page.tsx

@@ -2,6 +2,8 @@ import { getDictionary } from "@/app/[lang]/dictionaries";
 import { getInvite } from "@/models/invite";
 import { HydrationBoundary, QueryClient, dehydrate } from "@tanstack/react-query";
 import InviteClient from "./invite-client";
+import { getCurrentUser } from "@/lib";
+import { User } from "@/models/user";
 
 export default async function Page({ params }: { params: Promise<{ lang: string; code: string }> }) {
   const { lang, code } = await params;
@@ -13,9 +15,15 @@ export default async function Page({ params }: { params: Promise<{ lang: string;
     queryFn: () => getInvite(code),
   });
 
+
+  let user: User | undefined = undefined;
+  try {
+    user = await getCurrentUser(false);
+  } catch (_) {}
+
   return (
     <HydrationBoundary state={dehydrate(queryClient)}>
-      <InviteClient code={code} dict={dict} />
+      <InviteClient code={code} dict={dict} currentUser={user} />
     </HydrationBoundary>
   );
 }

frontend-nextjs/src/app/[lang]/dashboard/organisation/[orgId]/(home)/dashboard.tsx

@@ -6,7 +6,7 @@ import { useQuery } from "@tanstack/react-query";
 export default function Dashboard() {
     const { data } = useQuery({
         queryKey: ['user'],
-        queryFn: getCurrentUser,
+        queryFn: () => getCurrentUser(),
     });
 
     return (

frontend-nextjs/src/app/[lang]/dashboard/organisation/[orgId]/campaigns/[campaignId]/campaign-details.tsx

@@ -124,7 +124,7 @@ export default function CampaignDetails({ campaignId, orgId, dict }: { campaignI
                         </Link>
                     </ButtonGroup>
                     <ButtonGroup>
-                        <CopyButton value={`${process.env.NEXT_PUBLIC_APP_URL}/campaign/${campaign?.organisation_slug}/${campaign?.campaign_slug}`}>
+                        <CopyButton value={`https://chaos.devsoc.app/campaign/${campaign?.organisation_slug}/${campaign?.campaign_slug}`}>
                             <Share className="w-4 h-4" /> {dict.dashboard.campaigns.share_link}
                         </CopyButton>
                         <CopyButton value={campaignId}>