From f9f3501c6cc5a826ec412de231604814d024f01f Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sun, 1 Feb 2026 15:15:12 +0530
Subject: [PATCH 01/11] feat: add disabled prop to Avatar component

---
 frontend/src/components/ui/Avatar.vue | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/frontend/src/components/ui/Avatar.vue b/frontend/src/components/ui/Avatar.vue
index 4160d04..700cb25 100644
--- a/frontend/src/components/ui/Avatar.vue
+++ b/frontend/src/components/ui/Avatar.vue
@@ -11,11 +11,13 @@ const props = withDefaults(
         variant?: "default" | "outline" | "filled";
         shape?: "circle" | "square";
         className?: string;
+        disabled?: boolean;
     }>(),
     {
         size: "md",
         variant: "default",
         shape: "circle",
+        disabled: false,
     }
 );
 
@@ -29,11 +31,16 @@ onMounted(async () => {
 </script>
 
 <template>
-    <Tooltip v-if="user?.full_name" :text="user?.full_name" placement="bottom">
+    <Tooltip
+        v-if="user?.full_name"
+        :disabled="disabled"
+        :text="user?.full_name"
+        placement="bottom"
+    >
         <FrappeAvatar
             :size="size"
             :shape="shape"
-            :image="user?.user_image"
+            :image="user?.user_image ?? undefined"
             :label="user?.full_name"
         />
     </Tooltip>

From 727df85bcf4380f05f9efffd175f9c1d55002be8 Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sat, 7 Feb 2026 21:07:52 +0530
Subject: [PATCH 02/11] chore: new sidebar

---
 forms_pro/api/team.py                |   1 -
 forms_pro/public/images/logo_300.svg |   5 ++
 frontend/src/layouts/BaseLayout.vue  | 103 +++++++++++++++++++--------
 3 files changed, 79 insertions(+), 30 deletions(-)
 create mode 100644 forms_pro/public/images/logo_300.svg

diff --git a/forms_pro/api/team.py b/forms_pro/api/team.py
index 58bfa09..a6aeeaa 100644
--- a/forms_pro/api/team.py
+++ b/forms_pro/api/team.py
@@ -35,7 +35,6 @@ def get_team_members(team_id: str) -> list[GetTeamMembersResponse]:
         user=frappe.session.user,
         throw=True,
     )
-    print("team_id", team_id)
 
     team: FPTeam = frappe.get_doc("FP Team", team_id)
     members = team.team_members
diff --git a/forms_pro/public/images/logo_300.svg b/forms_pro/public/images/logo_300.svg
new file mode 100644
index 0000000..de2b2ed
--- /dev/null
+++ b/forms_pro/public/images/logo_300.svg
@@ -0,0 +1,5 @@
+<svg width="300" height="300" viewBox="0 0 300 300" fill="none" xmlns="http://www.w3.org/2000/svg">
+<rect width="300" height="300" fill="white"/>
+<path d="M192.672 229.811C189.477 229.811 186.717 228.795 184.394 226.762C182.215 224.583 181.126 221.751 181.126 218.265C181.126 214.925 182.215 212.238 184.394 210.205C186.717 208.027 189.477 206.937 192.672 206.937C195.577 206.937 198.191 208.027 200.514 210.205C202.838 212.238 204 214.925 204 218.265C204 221.751 202.838 224.583 200.514 226.762C198.191 228.795 195.577 229.811 192.672 229.811Z" fill="black"/>
+<path d="M98.2677 227.85C96.0892 227.85 95 227.052 95 225.454C95 224.147 95.944 223.276 97.832 222.84L102.843 221.969C105.892 221.388 107.853 220.589 108.724 219.572C109.741 218.556 110.249 216.522 110.249 213.473V85.378C110.249 82.3281 109.741 80.2948 108.724 79.2782C107.853 78.2616 105.892 77.4628 102.843 76.8819L97.832 76.0105C95.944 75.5748 95 74.7034 95 73.3963C95 71.7988 96.0892 71 98.2677 71H168.415C171.755 71 173.57 72.6702 173.861 76.0105L175.604 112.173C175.894 114.642 174.95 115.877 172.772 115.877C171.029 115.877 170.012 114.933 169.722 113.045L168.633 105.856C167.471 98.3036 165.437 92.4217 162.533 88.21C159.628 83.9983 156.36 81.021 152.73 79.2782C149.099 77.3902 145.613 76.4462 142.273 76.4462C135.883 76.4462 131.526 77.4628 129.202 79.4961C126.878 81.5293 125.717 84.6518 125.717 88.8635V140.276C125.717 142.309 126.733 143.326 128.766 143.326H136.391C142.781 143.326 146.703 139.913 148.155 133.087L150.551 122.194C150.987 120.306 152.076 119.362 153.819 119.362C155.562 119.362 156.433 120.597 156.433 123.066V172.081C156.433 174.55 155.562 175.785 153.819 175.785C152.076 175.785 150.987 174.841 150.551 172.953L148.155 162.278C147.138 157.195 145.686 153.855 143.798 152.257C142.055 150.66 139.514 149.861 136.173 149.861H128.766C126.733 149.861 125.717 150.878 125.717 152.911V212.383C125.717 215.433 126.152 217.612 127.024 218.919C128.04 220.081 130.074 220.879 133.123 221.315L143.362 222.84C144.96 223.13 145.759 223.929 145.759 225.236C145.759 226.979 144.669 227.85 142.491 227.85H98.2677Z" fill="black"/>
+</svg>
diff --git a/frontend/src/layouts/BaseLayout.vue b/frontend/src/layouts/BaseLayout.vue
index 3f1bf83..b575a8d 100644
--- a/frontend/src/layouts/BaseLayout.vue
+++ b/frontend/src/layouts/BaseLayout.vue
@@ -1,30 +1,62 @@
 <template>
     <div class="flex h-screen w-full">
-        <Sidebar
-            :header="{
-                title: sidebarHeader.title,
-                subtitle: sidebarHeader.subtitle,
-                menuItems: sidebarHeader.menuItems,
-            }"
-            :sections="sidebarSections"
-        >
-            <template #header-logo>
-                <div
-                    class="font-instrument font-bold bg-surface-white rounded border border-outline-gray-1 py-[2px]"
-                >
-                    F.
-                </div>
+        <Sidebar :header="sidebarHeader" :sections="sidebarSections">
+            <template #footer-items="{ isCollapsed }">
+                <Popover placement="top-start">
+                    <template #target="{ togglePopover, isOpen }">
+                        <div
+                            @click="togglePopover"
+                            class="p-2 z-10 rounded-md flex gap-2 items-center hover:bg-surface-white cursor-pointer duration-200 ease-[cubic-bezier(0.4, 0, 0.2, 1)]"
+                            :class="{ 'bg-surface-white': isOpen, '!p-1': isCollapsed }"
+                        >
+                            <Avatar
+                                :disabled="true"
+                                v-if="session.user"
+                                :user-id="session.user"
+                                size="md"
+                                shape="square"
+                            />
+                            <div v-if="!isCollapsed" class="flex items-center gap-2">
+                                <div class="flex flex-col gap-1">
+                                    <span class="text-sm font-medium text-ink-gray-8">
+                                        {{ session.full_name }}
+                                    </span>
+                                    <span class="text-xs text-ink-gray-5">{{ session.user }}</span>
+                                </div>
+                                <ChevronsUpDown class="size-4 text-ink-gray-6" />
+                            </div>
+                        </div>
+                    </template>
+                    <template #body>
+                        <div
+                            class="flex flex-col gap-2 bg-surface-white shadow-2xl z-0 rounded-t-lg -mb-2 p-2 pb-4 w-full"
+                            :class="{ 'rounded-br-lg': isCollapsed }"
+                        >
+                            <Button
+                                variant="ghost"
+                                theme="red"
+                                @click="session.logout.submit"
+                                label="Log out"
+                                icon-left="log-out"
+                            />
+                        </div>
+                    </template>
+                </Popover>
             </template>
         </Sidebar>
-        <slot></slot>
+        <slot> </slot>
     </div>
 </template>
 <script setup lang="ts">
 import { session } from "@/data/session";
 import { computed } from "vue";
-import { Sidebar, type SidebarProps } from "frappe-ui";
-import { LayoutDashboard, LogOut } from "lucide-vue-next";
+import { Popover, Sidebar, type SidebarProps } from "frappe-ui";
+import { ChevronsUpDown, LayoutDashboard, Plus } from "lucide-vue-next";
 import type { PropType } from "vue";
+import Avatar from "@/components/ui/Avatar.vue";
+import { useUser } from "@/stores/user";
+
+const userStore = useUser();
 
 type SidebarSectionProps = NonNullable<SidebarProps["sections"]> extends (infer T)[] ? T : never;
 type SidebarHeaderProps = NonNullable<SidebarProps["header"]>;
@@ -40,24 +72,37 @@ const props = defineProps({
     },
 });
 
-const defaultSidebarMenuItems = computed(() => [
-    {
-        label: "Log out",
-        icon: LogOut,
-        onClick: session.logout.submit,
-    },
-]);
-
 const sidebarHeader = computed(() => {
     if (props.sidebarHeader) {
         return props.sidebarHeader;
     }
 
+    const switchTeamItems = userStore.userTeams
+        ?.filter((team) => team.name !== userStore.currentTeam?.name)
+        .map((team) => ({
+            label: `Switch to ${team.team_name}`,
+            onClick: () => {
+                console.log("Change team to", team.team_name);
+            },
+        }));
+
+    const menuItems = [
+        ...switchTeamItems,
+        {
+            label: "Create New Team",
+            icon: Plus,
+            onClick: () => {
+                console.log("Create new team");
+            },
+        },
+    ];
+
     return {
-        title: "Forms Pro",
-        subtitle: session.full_name ?? undefined,
-        menuItems: defaultSidebarMenuItems.value,
-    };
+        title: userStore.currentTeam?.team_name || "",
+        subtitle: "Switch Teams",
+        logo: "/assets/forms_pro/images/logo_300.svg",
+        menuItems: menuItems,
+    } as SidebarHeaderProps;
 });
 
 const sidebarSections = computed((): SidebarSectionProps[] => {

From c81f29bc15d4373b4f9f8feaeb0a225ba2137fab Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sat, 7 Feb 2026 21:10:19 +0530
Subject: [PATCH 03/11] style: update Avatar size and text styling in
 BaseLayout

---
 frontend/src/layouts/BaseLayout.vue | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/frontend/src/layouts/BaseLayout.vue b/frontend/src/layouts/BaseLayout.vue
index b575a8d..ae580a5 100644
--- a/frontend/src/layouts/BaseLayout.vue
+++ b/frontend/src/layouts/BaseLayout.vue
@@ -13,12 +13,12 @@
                                 :disabled="true"
                                 v-if="session.user"
                                 :user-id="session.user"
-                                size="md"
+                                size="lg"
                                 shape="square"
                             />
                             <div v-if="!isCollapsed" class="flex items-center gap-2">
                                 <div class="flex flex-col gap-1">
-                                    <span class="text-sm font-medium text-ink-gray-8">
+                                    <span class="text-base font-medium text-ink-gray-7">
                                         {{ session.full_name }}
                                     </span>
                                     <span class="text-xs text-ink-gray-5">{{ session.user }}</span>

From d9d4971f1737f2cfa89f0e837a791f232c8037ca Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sun, 8 Feb 2026 01:30:35 +0530
Subject: [PATCH 04/11] feat: implement team creation and switching
 functionality

- Added create_team and switch_team API methods to manage team operations.
- Introduced CreateTeamDialog component for user interface to create new teams.
- Updated user store to handle team creation and switching with success/error notifications.
- Enhanced FPTeam document to automatically set the current team upon creation.
---
 forms_pro/api/team.py                         | 37 ++++++++-
 .../forms_pro/doctype/fp_team/fp_team.py      | 30 +++++++-
 forms_pro/overrides/roles.py                  |  3 +-
 .../src/components/team/CreateTeamDialog.vue  | 77 +++++++++++++++++++
 frontend/src/layouts/BaseLayout.vue           | 10 ++-
 frontend/src/stores/team.ts                   |  6 +-
 frontend/src/stores/user.ts                   | 66 +++++++++++++++-
 7 files changed, 217 insertions(+), 12 deletions(-)
 create mode 100644 frontend/src/components/team/CreateTeamDialog.vue

diff --git a/forms_pro/api/team.py b/forms_pro/api/team.py
index a6aeeaa..bcc8f33 100644
--- a/forms_pro/api/team.py
+++ b/forms_pro/api/team.py
@@ -1,7 +1,7 @@
 import frappe
 
 from forms_pro.forms_pro.doctype.fp_team.fp_team import FPTeam, GetTeamMembersResponse
-from forms_pro.utils.teams import GetTeamFormsResponseSchema
+from forms_pro.utils.teams import GetTeamFormsResponseSchema, set_current_team
 from forms_pro.utils.teams import get_team_forms as get_team_forms_utils
 
 
@@ -40,3 +40,38 @@ def get_team_members(team_id: str) -> list[GetTeamMembersResponse]:
     members = team.team_members
 
     return members
+
+
+@frappe.whitelist(methods=["POST"])
+def create_team(team_name: str) -> FPTeam:
+    """
+    Create a new team
+
+    Args:
+        team_name: Name of the team
+
+    Returns:
+        FPTeam - The created team
+    """
+
+    team: FPTeam = frappe.new_doc("FP Team")
+    team.team_name = team_name
+    team.insert()
+    return team
+
+
+@frappe.whitelist(methods=["POST"])
+def switch_team(team_id: str) -> None:
+    """
+    Switch to a new team
+    """
+
+    if not frappe.has_permission(
+        doctype="FP Team",
+        ptype="read",
+        doc=team_id,
+        user=frappe.session.user,
+    ):
+        raise frappe.PermissionError("You do not have permission to switch to this team")
+
+    set_current_team(team_id, frappe.session.user)
diff --git a/forms_pro/forms_pro/doctype/fp_team/fp_team.py b/forms_pro/forms_pro/doctype/fp_team/fp_team.py
index e717cb8..6e812e4 100644
--- a/forms_pro/forms_pro/doctype/fp_team/fp_team.py
+++ b/forms_pro/forms_pro/doctype/fp_team/fp_team.py
@@ -1,12 +1,14 @@
 # Copyright (c) 2025, harsh@buildwithhussain.com and contributors
 # For license information, please see license.txt
 
-# import frappe
+import frappe
 from frappe.model.document import Document
+from frappe.share import add_docshare
 from frappe.utils import cached_property
 from pydantic import BaseModel, EmailStr
 
 from forms_pro.api.user import get_user
+from forms_pro.utils.teams import set_current_team
 
 
 class GetTeamMembersResponse(BaseModel):
@@ -61,3 +63,29 @@ def is_team_member(self, user: str) -> bool:
             bool - True if the user is a member of the team, False otherwise
         """
         return user in self.team_members
+
+    def after_insert(self) -> None:
+        self.add_to_team(self.owner)
+        set_current_team(self.name, self.owner)
+        self.save()
+
+    def add_to_team(self, user: str) -> None:
+        """
+        Add a user to the team
+
+        Args:
+            user: The user email address
+        """
+        if user == "Administrator":
+            frappe.throw("Administrator cannot be added to a team")
+
+        self.append("users", {"user": user})
+        add_docshare(
+            self.doctype,
+            self.name,
+            user,
+            read=1,
+            write=1,
+            share=1,
+            flags={"ignore_share_permission": True},
+        )
diff --git a/forms_pro/overrides/roles.py b/forms_pro/overrides/roles.py
index 8655789..fd73088 100644
--- a/forms_pro/overrides/roles.py
+++ b/forms_pro/overrides/roles.py
@@ -2,7 +2,7 @@
 from frappe.core.doctype.user.user import User
 
 from forms_pro.roles import FORMS_PRO_ROLE
-from forms_pro.utils.teams import get_user_teams, set_current_team
+from forms_pro.utils.teams import get_user_teams
 
 
 def has_forms_pro_permission() -> bool:
@@ -48,4 +48,3 @@ def create_default_team_for_user(user: User) -> None:
         },
     )
     team.save(ignore_permissions=True)
-    set_current_team(team.name, user.name)
diff --git a/frontend/src/components/team/CreateTeamDialog.vue b/frontend/src/components/team/CreateTeamDialog.vue
new file mode 100644
index 0000000..98f8027
--- /dev/null
+++ b/frontend/src/components/team/CreateTeamDialog.vue
@@ -0,0 +1,77 @@
+<script setup lang="ts">
+import { Dialog, ErrorMessage, FormControl } from "frappe-ui";
+import * as z from "zod";
+import { reactive, ref, watch } from "vue";
+import { useUser } from "@/stores/user";
+
+const user = useUser();
+
+const model = defineModel<boolean>({
+    type: Boolean,
+    required: true,
+});
+
+const formErrors = ref<string>("");
+
+const formSchema = z.object({
+    team_name: z
+        .string()
+        .min(2, "Team name must be at least 2 characters long")
+        .max(140, "Team name must be less than 140 characters long"),
+});
+
+type Form = z.infer<typeof formSchema>;
+
+const form = reactive<Form>({
+    team_name: "",
+});
+
+watch(
+    () => form,
+    (newVal) => {
+        const result = formSchema.safeParse(newVal);
+        if (!result.success) {
+            formErrors.value = result.error.issues.map((issue) => issue.message).join(", ");
+        } else {
+            formErrors.value = "";
+        }
+    },
+    { deep: true }
+);
+
+function createTeam() {
+    const result = formSchema.safeParse(form);
+    if (!result.success) {
+        formErrors.value = result.error.issues.map((issue) => issue.message).join(", ");
+    } else {
+        formErrors.value = "";
+    }
+    user.createTeam(form.team_name);
+    model.value = false;
+}
+</script>
+<template>
+    <Dialog
+        v-model="model"
+        :options="{
+            title: 'Create New Team',
+            icon: 'users',
+        }"
+    >
+        <template #body-content>
+            <div class="flex flex-col gap-4">
+                <FormControl
+                    v-model="form.team_name"
+                    label="Team Name"
+                    variant="outline"
+                    type="text"
+                    required
+                />
+                <ErrorMessage :message="formErrors" />
+                <Button @click="createTeam" :disabled="formErrors !== ''" variant="subtle">
+                    Create Team
+                </Button>
+            </div>
+        </template>
+    </Dialog>
+</template>
diff --git a/frontend/src/layouts/BaseLayout.vue b/frontend/src/layouts/BaseLayout.vue
index ae580a5..933d1f1 100644
--- a/frontend/src/layouts/BaseLayout.vue
+++ b/frontend/src/layouts/BaseLayout.vue
@@ -1,5 +1,6 @@
 <template>
     <div class="flex h-screen w-full">
+        <CreateTeamDialog v-model="showCreateTeamDialog" />
         <Sidebar :header="sidebarHeader" :sections="sidebarSections">
             <template #footer-items="{ isCollapsed }">
                 <Popover placement="top-start">
@@ -49,18 +50,21 @@
 </template>
 <script setup lang="ts">
 import { session } from "@/data/session";
-import { computed } from "vue";
+import { computed, ref } from "vue";
 import { Popover, Sidebar, type SidebarProps } from "frappe-ui";
 import { ChevronsUpDown, LayoutDashboard, Plus } from "lucide-vue-next";
 import type { PropType } from "vue";
 import Avatar from "@/components/ui/Avatar.vue";
 import { useUser } from "@/stores/user";
+import CreateTeamDialog from "@/components/team/CreateTeamDialog.vue";
 
 const userStore = useUser();
 
 type SidebarSectionProps = NonNullable<SidebarProps["sections"]> extends (infer T)[] ? T : never;
 type SidebarHeaderProps = NonNullable<SidebarProps["header"]>;
 
+const showCreateTeamDialog = ref(false);
+
 const props = defineProps({
     sidebarHeader: {
         type: Object as PropType<SidebarHeaderProps | null>,
@@ -82,7 +86,7 @@ const sidebarHeader = computed(() => {
         .map((team) => ({
             label: `Switch to ${team.team_name}`,
             onClick: () => {
-                console.log("Change team to", team.team_name);
+                userStore.switchTeam(team);
             },
         }));
 
@@ -92,7 +96,7 @@ const sidebarHeader = computed(() => {
             label: "Create New Team",
             icon: Plus,
             onClick: () => {
-                console.log("Create new team");
+                showCreateTeamDialog.value = true;
             },
         },
     ];
diff --git a/frontend/src/stores/team.ts b/frontend/src/stores/team.ts
index 79dd1b9..0f45e7a 100644
--- a/frontend/src/stores/team.ts
+++ b/frontend/src/stores/team.ts
@@ -1,6 +1,6 @@
 import { defineStore } from "pinia";
 import { useUser } from "./user";
-import { computed } from "vue";
+import { computed, watch } from "vue";
 import { useCall } from "frappe-ui";
 
 export type TeamMember = {
@@ -31,6 +31,10 @@ export const useTeam = defineStore("team", () => {
     teamMembersResource.fetch();
   }
 
+  watch(currentTeam, () => {
+    teamMembersResource.fetch();
+  });
+
   return {
     currentTeam,
     teamMembers,
diff --git a/frontend/src/stores/user.ts b/frontend/src/stores/user.ts
index 0d0565c..725cdf6 100644
--- a/frontend/src/stores/user.ts
+++ b/frontend/src/stores/user.ts
@@ -3,6 +3,7 @@ import setTheme from "@/utils/theme";
 import { createResource } from "frappe-ui";
 import { defineStore } from "pinia";
 import { computed, ref } from "vue";
+import { toast } from "vue-sonner";
 
 export type UserTeam = {
   name: string;
@@ -10,6 +11,11 @@ export type UserTeam = {
   is_current: boolean;
 };
 
+export type CreateTeamResponse = {
+  name: string;
+  team_name: string;
+};
+
 export const useUser = defineStore("user", () => {
   const user = computed(() => userResource.data);
   const userTeams = computed<UserTeam[]>(() => userTeamsResource.data);
@@ -21,15 +27,23 @@ export const useUser = defineStore("user", () => {
 
   const userTeamsResource = createResource({
     url: "forms_pro.api.user.get_user_teams",
+    onSuccess(data: UserTeam[]) {
+      const _currTeam = data.find((team) => team.is_current);
+      if (!_currTeam) {
+        return;
+      }
+      setCurrentTeam(_currTeam);
+    },
+    onError(error: Error) {
+      toast.error("Failed to fetch user teams", {
+        description: error.message,
+      });
+    },
   });
 
   async function initialize() {
     await userResource.fetch();
     await userTeamsResource.fetch();
-    const _currTeam = getCurrentTeamFromAllTeams();
-    if (_currTeam) {
-      setCurrentTeam(_currTeam);
-    }
   }
 
   function fetchUser() {
@@ -48,6 +62,48 @@ export const useUser = defineStore("user", () => {
     currentTeam.value = team;
   }
 
+  function switchTeam(team: UserTeam) {
+    createResource({
+      url: "forms_pro.api.team.switch_team",
+      method: "POST",
+      makeParams() {
+        return {
+          team_id: team.name,
+        };
+      },
+      onSuccess() {
+        userTeamsResource.reload();
+        toast.success("Team switched successfully");
+      },
+      onError(error: Error) {
+        toast.error("Failed to switch team", {
+          description: error.message,
+        });
+      },
+    }).submit();
+  }
+
+  function createTeam(teamName: string) {
+    createResource({
+      url: "forms_pro.api.team.create_team",
+      method: "POST",
+      makeParams() {
+        return {
+          team_name: teamName,
+        };
+      },
+      onSuccess(data: CreateTeamResponse) {
+        userTeamsResource.reload();
+        toast.success("Team created successfully");
+      },
+      onError(error: Error) {
+        toast.error("Failed to create team", {
+          description: error.message,
+        });
+      },
+    }).submit();
+  }
+
   async function toggleThemePreference(theme: ThemePreferenceType) {
     setTheme(theme);
 
@@ -70,6 +126,8 @@ export const useUser = defineStore("user", () => {
     fetchUser,
     fetchUserTeams,
     setCurrentTeam,
+    createTeam,
+    switchTeam,
     toggleThemePreference,
   };
 });

From fb1598b89361922e30431b5a9f03a08378e16eb4 Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sun, 8 Feb 2026 01:39:46 +0530
Subject: [PATCH 05/11] fix: ensure switch team items are defined in BaseLayout

---
 frontend/src/layouts/BaseLayout.vue | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/frontend/src/layouts/BaseLayout.vue b/frontend/src/layouts/BaseLayout.vue
index 933d1f1..3b8a969 100644
--- a/frontend/src/layouts/BaseLayout.vue
+++ b/frontend/src/layouts/BaseLayout.vue
@@ -81,14 +81,15 @@ const sidebarHeader = computed(() => {
         return props.sidebarHeader;
     }
 
-    const switchTeamItems = userStore.userTeams
-        ?.filter((team) => team.name !== userStore.currentTeam?.name)
-        .map((team) => ({
-            label: `Switch to ${team.team_name}`,
-            onClick: () => {
-                userStore.switchTeam(team);
-            },
-        }));
+    const switchTeamItems =
+        userStore.userTeams
+            ?.filter((team) => team.name !== userStore.currentTeam?.name)
+            .map((team) => ({
+                label: `Switch to ${team.team_name}`,
+                onClick: () => {
+                    userStore.switchTeam(team);
+                },
+            })) ?? [];
 
     const menuItems = [
         ...switchTeamItems,

From d6eecfa546dfb93bc607c0757e258e43a2176972 Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sun, 8 Feb 2026 02:01:05 +0530
Subject: [PATCH 06/11] chore: add tests

---
 .../forms_pro/doctype/fp_team/fp_team.py      |  5 +-
 .../forms_pro/doctype/fp_team/test_fp_team.py | 57 ++++++++++++++++++-
 forms_pro/install.py                          |  7 +--
 forms_pro/tests/__init__.py                   |  2 +
 4 files changed, 62 insertions(+), 9 deletions(-)
 create mode 100644 forms_pro/tests/__init__.py

diff --git a/forms_pro/forms_pro/doctype/fp_team/fp_team.py b/forms_pro/forms_pro/doctype/fp_team/fp_team.py
index 6e812e4..9ef4e3f 100644
--- a/forms_pro/forms_pro/doctype/fp_team/fp_team.py
+++ b/forms_pro/forms_pro/doctype/fp_team/fp_team.py
@@ -1,7 +1,6 @@
 # Copyright (c) 2025, harsh@buildwithhussain.com and contributors
 # For license information, please see license.txt
 
-import frappe
 from frappe.model.document import Document
 from frappe.share import add_docshare
 from frappe.utils import cached_property
@@ -62,7 +61,7 @@ def is_team_member(self, user: str) -> bool:
         Returns:
             bool - True if the user is a member of the team, False otherwise
         """
-        return user in self.team_members
+        return user in [member["email"] for member in self.team_members]
 
     def after_insert(self) -> None:
         self.add_to_team(self.owner)
@@ -77,7 +76,7 @@ def add_to_team(self, user: str) -> None:
             user: The user email address
         """
         if user == "Administrator":
-            frappe.throw("Administrator cannot be added to a team")
+            return
 
         self.append("users", {"user": user})
         add_docshare(
diff --git a/forms_pro/forms_pro/doctype/fp_team/test_fp_team.py b/forms_pro/forms_pro/doctype/fp_team/test_fp_team.py
index e353069..7551de5 100644
--- a/forms_pro/forms_pro/doctype/fp_team/test_fp_team.py
+++ b/forms_pro/forms_pro/doctype/fp_team/test_fp_team.py
@@ -1,9 +1,12 @@
 # Copyright (c) 2025, harsh@buildwithhussain.com and Contributors
 # See license.txt
 
-# import frappe
+import frappe
+from frappe.defaults import get_user_default
 from frappe.tests import IntegrationTestCase
 
+from forms_pro.tests import FORMS_PRO_TEST_USER
+
 # On IntegrationTestCase, the doctype test records and all
 # link-field test record dependencies are recursively loaded
 # Use these module variables to add/remove to/from that list
@@ -17,4 +20,54 @@ class IntegrationTestFPTeam(IntegrationTestCase):
     Use this class for testing interactions between multiple components.
     """
 
-    pass
+    def setUp(self):
+        super().setUp()
+        self.test_user = FORMS_PRO_TEST_USER
+
+    def test_add_owner_to_team(self):
+        """
+        Test that after a user creates a team, that owner user is added to the team and the team is shared with the owner user
+        """
+
+        frappe.set_user(self.test_user)
+
+        team = frappe.new_doc("FP Team")
+        team.team_name = "Test Team"
+        team.insert()
+        team.reload()
+
+        frappe.set_user("Administrator")
+
+        # Check that the user is added to the team
+        self.assertTrue(team.is_team_member(self.test_user))
+
+        # Check that the user is added to the team
+        self.assertIsNotNone(
+            frappe.db.exists(
+                "FP Team Member",
+                {
+                    "parent": team.name,
+                    "parentfield": "users",
+                    "parenttype": "FP Team",
+                    "user": self.test_user,
+                },
+            )
+        )
+
+        # Check that the user is added to the team's docshare
+        self.assertTrue(
+            frappe.db.exists(
+                "DocShare",
+                {
+                    "share_doctype": "FP Team",
+                    "share_name": team.name,
+                    "user": self.test_user,
+                    "read": 1,
+                    "write": 1,
+                    "share": 1,
+                },
+            )
+        )
+
+        # Check that the user's current team is set to the team
+        self.assertEqual(get_user_default("current_team", self.test_user), team.name)
diff --git a/forms_pro/install.py b/forms_pro/install.py
index e719601..f408d42 100644
--- a/forms_pro/install.py
+++ b/forms_pro/install.py
@@ -2,6 +2,7 @@
 from frappe.core.doctype.user.user import User
 
 from forms_pro.roles import FORMS_PRO_ROLE
+from forms_pro.tests import FORMS_PRO_TEST_USER
 
 
 def after_install():
@@ -30,13 +31,11 @@ def give_admin_forms_pro_role():
 
 
 def create_test_user():
-    test_user = "test_forms_pro_user@example.com"
-
-    if frappe.db.exists("User", test_user):
+    if frappe.db.exists("User", FORMS_PRO_TEST_USER):
         return
 
     user: User = frappe.new_doc("User")
-    user.email = test_user
+    user.email = FORMS_PRO_TEST_USER
     user.first_name = "Test"
     user.last_name = "Forms Pro User"
     user.insert(ignore_permissions=True)
diff --git a/forms_pro/tests/__init__.py b/forms_pro/tests/__init__.py
new file mode 100644
index 0000000..1f90860
--- /dev/null
+++ b/forms_pro/tests/__init__.py
@@ -0,0 +1,2 @@
+# Test user email address
+FORMS_PRO_TEST_USER = "test_forms_pro_user@example.com"

From 227c27daf2d305fda149eab9620ee9f57c8804c0 Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sun, 8 Feb 2026 02:04:29 +0530
Subject: [PATCH 07/11] fix: prevent duplicate team membership

- Added a check to ensure that a user cannot be added to the team if they are already a member, throwing a DuplicateEntryError if the condition is met.
---
 forms_pro/forms_pro/doctype/fp_team/fp_team.py | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/forms_pro/forms_pro/doctype/fp_team/fp_team.py b/forms_pro/forms_pro/doctype/fp_team/fp_team.py
index 9ef4e3f..f9b4497 100644
--- a/forms_pro/forms_pro/doctype/fp_team/fp_team.py
+++ b/forms_pro/forms_pro/doctype/fp_team/fp_team.py
@@ -1,6 +1,7 @@
 # Copyright (c) 2025, harsh@buildwithhussain.com and contributors
 # For license information, please see license.txt
 
+import frappe
 from frappe.model.document import Document
 from frappe.share import add_docshare
 from frappe.utils import cached_property
@@ -78,6 +79,12 @@ def add_to_team(self, user: str) -> None:
         if user == "Administrator":
             return
 
+        if self.is_team_member(user):
+            frappe.throw(
+                frappe._("User is already a member of the team"),
+                frappe.DuplicateEntryError,
+            )
+
         self.append("users", {"user": user})
         add_docshare(
             self.doctype,

From 73eecf88ffd1707ed69d5b1fe962e9a5d662f652 Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sun, 8 Feb 2026 02:06:27 +0530
Subject: [PATCH 08/11] fix: handle form validation errors in CreateTeamDialog

- Updated createTeam function to return early on validation failure, preventing unnecessary team creation attempts.
- Ensured formErrors are cleared appropriately based on validation results.
---
 frontend/src/components/team/CreateTeamDialog.vue | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/frontend/src/components/team/CreateTeamDialog.vue b/frontend/src/components/team/CreateTeamDialog.vue
index 98f8027..f02e8ea 100644
--- a/frontend/src/components/team/CreateTeamDialog.vue
+++ b/frontend/src/components/team/CreateTeamDialog.vue
@@ -43,9 +43,9 @@ function createTeam() {
     const result = formSchema.safeParse(form);
     if (!result.success) {
         formErrors.value = result.error.issues.map((issue) => issue.message).join(", ");
-    } else {
-        formErrors.value = "";
+        return;
     }
+    formErrors.value = "";
     user.createTeam(form.team_name);
     model.value = false;
 }

From e827881a00d908875f18234efcbfedff542795ed Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sun, 8 Feb 2026 02:07:13 +0530
Subject: [PATCH 09/11] fix: set default current team if none is marked as
 current

- Added a check to set the first team as the current team if no current team is found in the user store.
---
 frontend/src/stores/user.ts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/frontend/src/stores/user.ts b/frontend/src/stores/user.ts
index 725cdf6..99090b2 100644
--- a/frontend/src/stores/user.ts
+++ b/frontend/src/stores/user.ts
@@ -30,6 +30,9 @@ export const useUser = defineStore("user", () => {
     onSuccess(data: UserTeam[]) {
       const _currTeam = data.find((team) => team.is_current);
       if (!_currTeam) {
+        if (data.length > 0) {
+          setCurrentTeam(data[0]);
+        }
         return;
       }
       setCurrentTeam(_currTeam);

From bd0e6d55a1e11ede010e3772ed834788c6973db8 Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sun, 8 Feb 2026 02:07:53 +0530
Subject: [PATCH 10/11] test: add tearDown method to IntegrationTestFPTeam for
 user reset

---
 forms_pro/forms_pro/doctype/fp_team/test_fp_team.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/forms_pro/forms_pro/doctype/fp_team/test_fp_team.py b/forms_pro/forms_pro/doctype/fp_team/test_fp_team.py
index 7551de5..20caaad 100644
--- a/forms_pro/forms_pro/doctype/fp_team/test_fp_team.py
+++ b/forms_pro/forms_pro/doctype/fp_team/test_fp_team.py
@@ -24,6 +24,10 @@ def setUp(self):
         super().setUp()
         self.test_user = FORMS_PRO_TEST_USER
 
+    def tearDown(self):
+        frappe.set_user("Administrator")
+        super().tearDown()
+
     def test_add_owner_to_team(self):
         """
         Test that after a user creates a team, that owner user is added to the team and the team is shared with the owner user

From 63dd63e3581e4b64336bb12eafe522162f9abadd Mon Sep 17 00:00:00 2001
From: Harsh Tandiya <harsh.tandiya@gmail.com>
Date: Sun, 8 Feb 2026 02:18:51 +0530
Subject: [PATCH 11/11] fix: flaxy test

---
 forms_pro/forms_pro/doctype/fp_team/fp_team.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/forms_pro/forms_pro/doctype/fp_team/fp_team.py b/forms_pro/forms_pro/doctype/fp_team/fp_team.py
index f9b4497..4ecc550 100644
--- a/forms_pro/forms_pro/doctype/fp_team/fp_team.py
+++ b/forms_pro/forms_pro/doctype/fp_team/fp_team.py
@@ -4,7 +4,6 @@
 import frappe
 from frappe.model.document import Document
 from frappe.share import add_docshare
-from frappe.utils import cached_property
 from pydantic import BaseModel, EmailStr
 
 from forms_pro.api.user import get_user
@@ -32,7 +31,7 @@ class FPTeam(Document):
         users: DF.TableMultiSelect[FPTeamMember]
     # end: auto-generated types
 
-    @cached_property
+    @property
     def team_members(self) -> list[GetTeamMembersResponse]:
         """
         Get the list of team members