fixed combo-box and merging new system tags

Author: rajat1saxena

apps/web/components/admin/products/new-customer.tsx

@@ -49,7 +49,11 @@ export default function NewCustomer({ courseId }: NewCustomerProps) {
                 setSystemTags(response.tags);
             }
         } catch (err) {
-        } finally {
+            toast({
+                title: TOAST_TITLE_ERROR,
+                description: err.message,
+                variant: "destructive",
+            });
         }
     }, [address.backend]);
 
@@ -92,6 +96,9 @@ export default function NewCustomer({ courseId }: NewCustomerProps) {
             if (response.user) {
                 setEmail("");
                 setTags([]);
+                if (response.user.tags) {
+                    mergeSubmittedTagsWithSystemTags(response.user.tags);
+                }
                 const message = `${response.user.email} has been invited.`;
                 toast({
                     title: TOAST_TITLE_SUCCESS,
@@ -108,6 +115,12 @@ export default function NewCustomer({ courseId }: NewCustomerProps) {
         }
     };
 
+    const mergeSubmittedTagsWithSystemTags = (submittedTags: string[]) => {
+        const fullSystemTags = [...systemTags, ...submittedTags];
+        const uniqueSystemTags = new Set(fullSystemTags);
+        setSystemTags(Array.from(uniqueSystemTags));
+    };
+
     return (
         <div className="flex flex-col gap-4">
             <>

packages/components-library/src/combo-box.tsx

@@ -13,7 +13,6 @@ interface ComboBoxProps {
     selectedOptions: Set<string>;
     onChange: (options: string[]) => void;
     className?: string;
-    side?: "left" | "right" | "top" | "bottom";
     placeholder?: string;
 }
 
@@ -22,7 +21,6 @@ export default function ComboBox({
     selectedOptions,
     onChange,
     className,
-    side = "top",
     placeholder = "Select an option or create one",
 }: ComboBoxProps) {
     const [internalOptions, setInternalOptions] = useState(options);
@@ -129,6 +127,7 @@ export default function ComboBox({
     return (
         <div className="relative w-full">
             <Button
+                type="button"
                 variant="outline"
                 role="combobox"
                 aria-expanded={internalOpen}