Skip to content

Commit d78975d

Browse files
authored
Merge pull request #2608 from RedisInsight/fe/bugfix/RI-4939_not_send_telemetry_event
#RI-4939, #RI-4934
2 parents 1229fa0 + 1b04895 commit d78975d

File tree

4 files changed

+17
-10
lines changed

4 files changed

+17
-10
lines changed

redisinsight/ui/src/components/oauth/oauth-select-plan/OAuthSelectPlan.tsx

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -161,12 +161,13 @@ const OAuthSelectPlan = () => {
161161
<EuiModal className={styles.container} onClose={handleOnClose} data-testid="oauth-select-plan-dialog">
162162
<EuiModalBody className={styles.modalBody}>
163163
<section className={styles.content}>
164-
<EuiText className={styles.subTitle}>
165-
Redis Enterprise Cloud
166-
</EuiText>
167164
<EuiTitle size="s">
168-
<h2 className={styles.title}>Select cloud vendor</h2>
165+
<h2 className={styles.title}>Choose a cloud vendor</h2>
169166
</EuiTitle>
167+
<EuiText className={styles.subTitle}>
168+
Select a cloud vendor and region to complete the final step towards
169+
your free Redis database. No credit card is required.
170+
</EuiText>
170171
<section className={styles.providers}>
171172
{ OAuthProviders.map(({ icon, id, label }) => (
172173
<div className={styles.provider} key={id}>

redisinsight/ui/src/components/oauth/oauth-select-plan/styles.module.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@
88
.title {
99
font-size: 28px !important;
1010
color: var(--htmlColor) !important;
11+
padding-top: 16px;
1112
}
1213

1314
.subTitle {
14-
padding-top: 24px;
15-
padding-bottom: 8px;
15+
padding: 24px 40px 0;
1616
font-size: 14px !important;
1717
}
1818

redisinsight/ui/src/components/oauth/oauth-sso-handler-dialog/OAuthSsoHandlerDialog.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,10 @@ import { appFeatureFlagsFeaturesSelector } from 'uiSrc/slices/app/features'
88
import { setSignInDialogState } from 'uiSrc/slices/oauth/cloud'
99

1010
export interface Props {
11-
children: (ssoCloudHandlerClick: (e: React.MouseEvent, source: OAuthSocialSource) => void) => React.ReactElement
11+
children: (
12+
ssoCloudHandlerClick: (e: React.MouseEvent, source: OAuthSocialSource) => void,
13+
isSSOEnabled: boolean,
14+
) => React.ReactElement
1215
}
1316

1417
const OAuthSsoHandlerDialog = ({ children }: Props) => {
@@ -32,7 +35,7 @@ const OAuthSsoHandlerDialog = ({ children }: Props) => {
3235
})
3336
}
3437

35-
return children?.(ssoCloudHandlerClick)
38+
return children?.(ssoCloudHandlerClick, !!feature?.flag)
3639
}
3740

3841
export default OAuthSsoHandlerDialog

redisinsight/ui/src/pages/home/components/HomeHeader/HomeHeader.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ const HomeHeader = ({ onAddInstance, direction }: Props) => {
185185
const linkStyles = stylesCss ? stylesCss[theme] : {}
186186
const promoLink = (
187187
<OAuthSsoHandlerDialog>
188-
{(ssoCloudHandlerClick) => (
188+
{(ssoCloudHandlerClick, isSSOEnabled) => (
189189
<PromoLink
190190
title={title}
191191
description={description}
@@ -199,7 +199,10 @@ const HomeHeader = ({ onAddInstance, direction }: Props) => {
199199
: undefined
200200
}}
201201
onClick={(e) => {
202-
handleCreateDatabaseClick(HELP_LINKS.cloud.event, { source: 'My Redis databases' })
202+
!isSSOEnabled && handleCreateDatabaseClick(
203+
HELP_LINKS.cloud.event,
204+
{ source: HELP_LINKS.cloud.sources.databaseList },
205+
)
203206
ssoCloudHandlerClick(e, OAuthSocialSource.ListOfDatabases)
204207
}}
205208
/>

0 commit comments

Comments
 (0)