checkbox for user to enable/disable email sharing (briefercloud#167)

* checkbox for user to enable/disable email sharing * Update apps/api/src/events/posthog.ts --------- Co-authored-by: Lucas Vieira <[email protected]>
DanFitzgibbon · Oct 25, 2024 · 616d331 · 616d331
1 parent 5930074
commit 616d331
Show file tree

Hide file tree

Showing 4 changed files with 41 additions and 5 deletions.
diff --git a/apps/api/src/auth/base.ts b/apps/api/src/auth/base.ts
@@ -15,6 +15,7 @@ import {
 } from './token.js'
 import { createWorkspace } from '../workspace/index.js'
 import { isWorkspaceNameValid } from '../utils/validation.js'
+import { captureWorkspaceCreated } from '../events/posthog.js'
 
 type BaseAuthConfig = {
   FRONTEND_URL: string
@@ -67,6 +68,7 @@ export default function getRouter<H extends ApiUser>(
         name: z.string().trim(),
         email: z.string().trim().email(),
         password: z.string(),
+        shareEmail: z.boolean(),
       })
       .safeParse(req.body)
 
@@ -77,7 +79,7 @@ export default function getRouter<H extends ApiUser>(
       return
     }
 
-    const { email, password } = payload.data
+    const { email, password, shareEmail } = payload.data
     if (!isValidPassword(password)) {
       res.status(400).json({
         reason: 'invalid-password',
@@ -115,6 +117,8 @@ export default function getRouter<H extends ApiUser>(
         return { workspace, user }
       })
 
+      captureWorkspaceCreated(user, workspace, shareEmail)
+
       const loginLink = createLoginLink(user.id, config.FRONTEND_URL)
 
       res.status(201).json({ workspace, loginLink })

diff --git a/apps/api/src/events/posthog.ts b/apps/api/src/events/posthog.ts
@@ -18,18 +18,17 @@ export function getPostHogClient() {
 
 export const captureWorkspaceCreated = async (
   sender: ApiUser,
-  workspace: ApiWorkspace
+  workspace: ApiWorkspace,
+  shareEmail: boolean
 ) => {
   const posthog = getPostHogClient()
   posthog?.capture({
     distinctId: sender.id,
     event: 'workspace_created',
     properties: {
       workspaceId: workspace.id,
-      workspaceUseContext: workspace.useContext,
-      workspaceUseCases: workspace.useCases,
-      workspaceSource: workspace.source,
       creatorId: sender.id,
+      ownerEmail: shareEmail ? sender.email : null,
     },
   })
 }

diff --git a/apps/web/src/components/setup/UserForm.tsx b/apps/web/src/components/setup/UserForm.tsx
@@ -7,13 +7,15 @@ import {
   UseFormRegister,
 } from 'react-hook-form'
 import Spin from '../Spin'
+import { Tooltip } from '../Tooltips'
 
 export type UserStepFormValues = {
   firstName: string
   lastName: string
   email: string
   password: string
   confirmPassword: string
+  shareEmail: boolean
 }
 
 interface Props {
@@ -100,6 +102,32 @@ function UserSetupForm(props: Props) {
                 })}
               />
             </div>
+            <div className="mt-2 flex items-center">
+              <input
+                {...props.register('shareEmail')}
+                id="shareEmail"
+                tabIndex={-1}
+                type="checkbox"
+                className="h-4 w-4 text-primary-600 transition duration-150 ease-in-out outline-0 ring-0 focus:ring-0 rounded-sm"
+              />
+              <span className="ml-2 text-sm text-gray-500 hover:cursor-pointer">
+                <label htmlFor="shareEmail">
+                  {`I'm okay to share my email with the Briefer team. `}
+                </label>
+                <Tooltip
+                  title="We will NOT send you spam, we promise."
+                  message="We use this field to understand who our users are and how we can improve Briefer."
+                  position="top"
+                  active
+                  className="inline-block"
+                  tooltipClassname="w-64"
+                >
+                  <span className="underline cursor-help hover:text-gray-600 flex gap-x-1.5 items-center">
+                    {'That will help us a lot!'}
+                  </span>
+                </Tooltip>
+              </span>
+            </div>
             <FormError msg={props.formErrors.email?.message} />
           </div>
         </div>

diff --git a/apps/web/src/pages/setup.tsx b/apps/web/src/pages/setup.tsx
@@ -54,6 +54,9 @@ function SetupForm() {
   } = useForm<UserStepFormValues>({
     mode: 'onSubmit',
     reValidateMode: 'onSubmit',
+    defaultValues: {
+      shareEmail: true,
+    },
   })
 
   const [formValues, setFormValues] = useState<
@@ -65,6 +68,7 @@ function SetupForm() {
     email: '',
     password: '',
     confirmPassword: '',
+    shareEmail: true,
   })
 
   const [currentStep, setCurrentStep] = useState<'workspace' | 'user'>(
@@ -94,6 +98,7 @@ function SetupForm() {
               name: `${payload.firstName} ${payload.lastName}`,
               email: payload.email,
               password: payload.password,
+              shareEmail: payload.shareEmail,
             }),
           }
         )