From 46d5ab4d71ec872432c2578a74788a7df094e440 Mon Sep 17 00:00:00 2001
From: Nick Diehl <47604184+ncdiehl11@users.noreply.github.com>
Date: Fri, 18 Oct 2024 13:33:19 -0400
Subject: [PATCH] fix(protocol-designer): update privacy settings copy (#16534)

Updates PD privacy settings copy according to latest legal
recommendations. Adds links within copy to direct to privacy policy and
opentrons EULA.

Closes AUTH-902
---
 .../src/assets/localization/en/shared.json    |  2 +-
 .../Settings/__tests__/Settings.test.tsx      |  6 ++--
 .../src/pages/Settings/index.tsx              | 32 +++++++++++++++----
 3 files changed, 30 insertions(+), 10 deletions(-)

diff --git a/protocol-designer/src/assets/localization/en/shared.json b/protocol-designer/src/assets/localization/en/shared.json
index 51525de47d6..737d92cc952 100644
--- a/protocol-designer/src/assets/localization/en/shared.json
+++ b/protocol-designer/src/assets/localization/en/shared.json
@@ -125,7 +125,7 @@
   "warning": "WARNING:",
   "wasteChute": "Waste chute",
   "wasteChuteAndStagingArea": "Waste chute and staging area slot",
-  "we_are_improving": "We’re working to improve Protocol Designer. Part of the process involves watching real user sessions to understand which parts of the interface are working and which could use improvement. We never share sessions outside of Opentrons.",
+  "we_are_improving": "In order to improve our products, Opentrons would like to collect data related to your use of Protocol Designer. With your consent, Opentrons will collect and store analytics and session data, including through the use of cookies and similar technologies, solely for the purpose enhancing our products. Find detailed information in our <link1>privacy policy</link1>. By using Protocol Designer, you consent to the Opentrons <link2>EULA</link2>.",
   "welcome": "Welcome to Protocol Designer!",
   "yes": "Yes"
 }
diff --git a/protocol-designer/src/pages/Settings/__tests__/Settings.test.tsx b/protocol-designer/src/pages/Settings/__tests__/Settings.test.tsx
index cb62cfe7d62..186b50f029c 100644
--- a/protocol-designer/src/pages/Settings/__tests__/Settings.test.tsx
+++ b/protocol-designer/src/pages/Settings/__tests__/Settings.test.tsx
@@ -53,9 +53,9 @@ describe('Settings', () => {
     screen.getByText('Reset hints')
     screen.getByText('Privacy')
     screen.getByText('Share sessions with Opentrons')
-    screen.getByText(
-      'We’re working to improve Protocol Designer. Part of the process involves watching real user sessions to understand which parts of the interface are working and which could use improvement. We never share sessions outside of Opentrons.'
-    )
+    screen.debug()
+    screen.getByRole('link', { name: 'privacy policy' })
+    screen.getByRole('link', { name: 'EULA' })
   })
   it('renders the announcement modal when view release notes button is clicked', () => {
     vi.mocked(AnnouncementModal).mockReturnValue(
diff --git a/protocol-designer/src/pages/Settings/index.tsx b/protocol-designer/src/pages/Settings/index.tsx
index 624cc4fa104..fc57ebc6d53 100644
--- a/protocol-designer/src/pages/Settings/index.tsx
+++ b/protocol-designer/src/pages/Settings/index.tsx
@@ -1,5 +1,5 @@
 import { useState } from 'react'
-import { useTranslation } from 'react-i18next'
+import { Trans, useTranslation } from 'react-i18next'
 import { useDispatch, useSelector } from 'react-redux'
 import { css } from 'styled-components'
 import {
@@ -11,6 +11,7 @@ import {
   Flex,
   Icon,
   JUSTIFY_SPACE_BETWEEN,
+  Link as LinkComponent,
   SPACING,
   StyledText,
   TYPOGRAPHY,
@@ -31,6 +32,8 @@ import { getFeatureFlagData } from '../../feature-flags/selectors'
 import type { FlagTypes } from '../../feature-flags'
 
 const HOT_KEY_FLAG = 'OT_PD_ENABLE_HOT_KEYS_DISPLAY'
+const PRIVACY_POLICY_URL = 'https://opentrons.com/privacy-policy'
+const EULA_URL = 'https://opentrons.com/eula'
 
 export function Settings(): JSX.Element {
   const dispatch = useDispatch()
@@ -235,11 +238,28 @@ export function Settings(): JSX.Element {
                   <StyledText desktopStyle="bodyDefaultSemiBold">
                     {t('shared:shared_sessions')}
                   </StyledText>
-                  <Flex color={COLORS.grey60}>
-                    <StyledText desktopStyle="bodyDefaultRegular">
-                      {t('shared:we_are_improving')}
-                    </StyledText>
-                  </Flex>
+                  <StyledText desktopStyle="bodyDefaultRegular">
+                    <Trans
+                      t={t}
+                      i18nKey="shared:we_are_improving"
+                      components={{
+                        link1: (
+                          <LinkComponent
+                            external
+                            href={PRIVACY_POLICY_URL}
+                            color={COLORS.blue50}
+                          />
+                        ),
+                        link2: (
+                          <LinkComponent
+                            external
+                            href={EULA_URL}
+                            color={COLORS.blue50}
+                          />
+                        ),
+                      }}
+                    />
+                  </StyledText>
                 </Flex>
                 <Btn
                   role="switch"