Skip to content

Commit ea23cfb

Browse files
committed
Merge remote-tracking branch 'origin/main' into feature/RI-6855/vector-search
2 parents 4580012 + 9a406e2 commit ea23cfb

File tree

7 files changed

+37
-25
lines changed

7 files changed

+37
-25
lines changed

redisinsight/ui/src/components/oauth/oauth-sign-in-button/OAuthSignInButton.tsx

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import React from 'react'
2+
import styled from 'styled-components'
23
import { OAuthSsoHandlerDialog } from 'uiSrc/components'
34

45
import RedisLogo from 'uiSrc/assets/img/logo_small.svg'
@@ -8,6 +9,11 @@ import { SecondaryButton } from 'uiSrc/components/base/forms/buttons'
89
import { RiImage } from 'uiSrc/components/base/display'
910
import styles from './styles.module.scss'
1011

12+
const LogoWrapper = styled.div`
13+
width: 15px;
14+
height: 15px;
15+
`
16+
1117
export interface Props {
1218
source: OAuthSocialSource
1319
}
@@ -29,7 +35,9 @@ const OAuthSignInButton = (props: Props) => {
2935
}
3036
data-testid="cloud-sign-in-btn"
3137
>
32-
<RiImage className={styles.logo} src={RedisLogo} alt="Redis logo" />
38+
<LogoWrapper>
39+
<RiImage $size={"fullWidth"} className={styles.logo} src={RedisLogo} alt="Redis logo" />
40+
</LogoWrapper>
3341
<span>Cloud sign in</span>
3442
</SecondaryButton>
3543
)}

redisinsight/ui/src/pages/browser/components/filter-key-type/FilterKeyType.tsx

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -23,18 +23,27 @@ import { appFeatureFlagsFeaturesSelector } from 'uiSrc/slices/app/features'
2323
import { AdditionalRedisModule } from 'uiSrc/slices/interfaces'
2424
import { OutsideClickDetector } from 'uiSrc/components/base/utils'
2525
import { HealthText } from 'uiSrc/components/base/text/HealthText'
26-
import { RiSelect } from 'uiSrc/components/base/forms/select/RiSelect'
26+
import {
27+
defaultValueRender,
28+
RiSelect,
29+
} from 'uiSrc/components/base/forms/select/RiSelect'
2730
import { Modal } from 'uiSrc/components/base/display'
2831
import { FILTER_KEY_TYPE_OPTIONS } from './constants'
2932

3033
import styles from './styles.module.scss'
34+
import styled from 'styled-components'
3135

3236
const ALL_KEY_TYPES_VALUE = 'all'
3337

3438
export interface Props {
3539
modules?: AdditionalRedisModule[]
3640
}
3741

42+
const FilterKeyTypeSelect = styled(RiSelect)`
43+
height: 100%;
44+
border-radius: 0;
45+
`
46+
3847
const FilterKeyType = ({ modules }: Props) => {
3948
const [isSelectOpen, setIsSelectOpen] = useState<boolean>(false)
4049
const [typeSelected, setTypeSelected] = useState<string>('all')
@@ -157,7 +166,9 @@ const FilterKeyType = ({ modules }: Props) => {
157166
onCancel={() => setIsInfoPopoverOpen(false)}
158167
className={styles.unsupportedInfoModal}
159168
data-testid="filter-not-available-modal"
160-
content={<FilterNotAvailable onClose={() => setIsInfoPopoverOpen(false)} />}
169+
content={
170+
<FilterNotAvailable onClose={() => setIsInfoPopoverOpen(false)} />
171+
}
161172
title={null}
162173
/>
163174
{!isVersionSupported && (
@@ -168,15 +179,10 @@ const FilterKeyType = ({ modules }: Props) => {
168179
data-testid="unsupported-btn-anchor"
169180
/>
170181
)}
171-
<RiSelect
182+
<FilterKeyTypeSelect
172183
disabled={!isVersionSupported}
173184
options={options}
174-
valueRender={({ option, isOptionValue }) => {
175-
if (isOptionValue) {
176-
return option.inputDisplay
177-
}
178-
return option.dropdownDisplay
179-
}}
185+
valueRender={defaultValueRender}
180186
defaultOpen={isSelectOpen}
181187
value={typeSelected}
182188
onChange={(value: string) => onChangeType(value)}

redisinsight/ui/src/pages/database-analysis/components/header/Header.spec.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ describe('DatabaseAnalysisHeader', () => {
140140

141141
const analizeButtonId = screen.getByTestId('start-database-analysis-btn')
142142
expect(analizeButtonId).toBeInTheDocument()
143-
expect(analizeButtonId).toHaveTextContent('Analyze')
143+
expect(analizeButtonId.textContent).toContain('New Report')
144144
})
145145

146146
it.skip('should call onChangeSelectedAnalysis after change selector', async () => {

redisinsight/ui/src/pages/pub-sub/components/messages-list/EmptyMessagesList/EmptyMessagesList.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@ import { ConnectionType } from 'uiSrc/slices/interfaces'
55
import { Text } from 'uiSrc/components/base/text'
66

77
import { RiIcon } from 'uiSrc/components/base/icons/RiIcon'
8-
import { Row } from 'uiSrc/components/base/layout/flex'
98
import styles from './styles.module.scss'
9+
import { Row } from 'uiSrc/components/base/layout/flex'
1010

1111
export interface Props {
1212
connectionType?: ConnectionType

redisinsight/ui/src/pages/rdi/pipeline-management/components/source-pipeline-dialog/SourcePipelineModal.spec.tsx

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,16 @@ jest.mock('uiSrc/telemetry', () => ({
3838
sendEventTelemetry: jest.fn(),
3939
}))
4040

41+
jest.mock('uiSrc/slices/rdi/pipeline', () => ({
42+
...jest.requireActual('uiSrc/slices/rdi/pipeline'),
43+
rdiPipelineSelector: jest.fn(),
44+
}))
45+
46+
jest.mock('uiSrc/slices/app/context', () => ({
47+
...jest.requireActual('uiSrc/slices/app/context'),
48+
appContextPipelineManagement: jest.fn(),
49+
}))
50+
4151
jest.mock('uiSrc/components/base/display', () => {
4252
const actual = jest.requireActual('uiSrc/components/base/display')
4353

redisinsight/ui/src/pages/rdi/pipeline-management/components/source-pipeline-dialog/SourcePipelineModal.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React, { useState } from 'react'
1+
import React, { useEffect, useState } from 'react'
22
import { keys } from '@elastic/eui'
33
import { useDispatch, useSelector } from 'react-redux'
44
import { useParams } from 'react-router-dom'

yarn.lock

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1560,18 +1560,6 @@
15601560
dependencies:
15611561
"@isaacs/balanced-match" "^4.0.1"
15621562

1563-
"@isaacs/balanced-match@^4.0.1":
1564-
version "4.0.1"
1565-
resolved "https://registry.yarnpkg.com/@isaacs/balanced-match/-/balanced-match-4.0.1.tgz#3081dadbc3460661b751e7591d7faea5df39dd29"
1566-
integrity sha512-yzMTt9lEb8Gv7zRioUilSglI0c0smZ9k5D65677DLWLtWJaXIS3CqcGyUFByYKlnUj6TkjLVs54fBl6+TiGQDQ==
1567-
1568-
"@isaacs/brace-expansion@^5.0.0":
1569-
version "5.0.0"
1570-
resolved "https://registry.yarnpkg.com/@isaacs/brace-expansion/-/brace-expansion-5.0.0.tgz#4b3dabab7d8e75a429414a96bd67bf4c1d13e0f3"
1571-
integrity sha512-ZT55BDLV0yv0RBm2czMiZ+SqCGO7AvmOM3G/w2xhVPH+te0aKgFjmBvGlL1dH+ql2tgGO3MVrbb3jCKyvpgnxA==
1572-
dependencies:
1573-
"@isaacs/balanced-match" "^4.0.1"
1574-
15751563
"@isaacs/cliui@^8.0.2":
15761564
version "8.0.2"
15771565
resolved "https://registry.yarnpkg.com/@isaacs/cliui/-/cliui-8.0.2.tgz#b37667b7bc181c168782259bab42474fbf52b550"

0 commit comments

Comments
 (0)