Skip to content

Commit b0efd87

Browse files
committed
fix: point connect modal docs links at cohort synchronisation docs
1 parent 3250d5f commit b0efd87

2 files changed

Lines changed: 7 additions & 4 deletions

File tree

frontend/web/components/modals/ConnectCohortProviderModal/ConnectCohortProviderModal.tsx

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ import {
1515
} from './providers'
1616
import './ConnectCohortProviderModal.scss'
1717

18-
const SEGMENTS_DOCS_URL = 'https://docs.flagsmith.com/basic-features/segments'
19-
2018
type ConnectCohortProviderModalProps = {
2119
projectId: number | string
2220
provider: CohortProviderKey
@@ -127,8 +125,8 @@ const ConnectCohortProviderModal: FC<ConnectCohortProviderModalProps> = ({
127125
</div>
128126
<ModalHR />
129127
<div className='modal-footer d-flex align-items-center justify-content-between'>
130-
<Link href={SEGMENTS_DOCS_URL} target='_blank'>
131-
Learn about Segments
128+
<Link href={config.docsUrl} target='_blank'>
129+
Learn about cohort synchronisation
132130
</Link>
133131
<Button onClick={() => closeModal()} data-test='connect-provider-done'>
134132
Done

frontend/web/components/modals/ConnectCohortProviderModal/providers.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ export type CohortProviderAuthRow = {
1111
export type CohortProviderConfig = {
1212
label: string
1313
authRows: CohortProviderAuthRow[]
14+
docsUrl: string
1415
// Providers where the user pastes a URL themselves carry an endpoint;
1516
// Amplitude calls Flagsmith's portal-registered endpoints instead.
1617
endpoint?: {
@@ -29,6 +30,8 @@ export const COHORT_PROVIDERS: Record<CohortProviderKey, CohortProviderConfig> =
2930
authRows: [
3031
{ label: 'API key', mono: true, value: '{YOUR_SYNCHRONISATION_KEY}' },
3132
],
33+
docsUrl:
34+
'https://docs.flagsmith.com/third-party-integrations/cohort-synchronisation/amplitude',
3235
endpointStepBody:
3336
'In Amplitude, open Data → Destinations and add Flagsmith as a cohort destination. Paste your synchronisation key when asked for the API key.',
3437
endpointStepTitle: 'Add Flagsmith as a destination in Amplitude',
@@ -43,6 +46,8 @@ export const COHORT_PROVIDERS: Record<CohortProviderKey, CohortProviderConfig> =
4346
{ label: 'Username', value: 'Any value' },
4447
{ label: 'Password', mono: true, value: '{YOUR_SYNCHRONISATION_KEY}' },
4548
],
49+
docsUrl:
50+
'https://docs.flagsmith.com/third-party-integrations/cohort-synchronisation',
4651
endpoint: {
4752
fieldTitle: 'Webhook URL',
4853
path: 'cohort-sync/mixpanel/webhook/',

0 commit comments

Comments
 (0)