Skip to content
This repository was archived by the owner on Apr 18, 2024. It is now read-only.

Commit 75118f2

Browse files
committed
chore: resolved mistakes by comments
1 parent b94fa08 commit 75118f2

File tree

11 files changed

+59
-34
lines changed

11 files changed

+59
-34
lines changed

src/common/components/containers/table.tsx

+14-4
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,27 @@
11
import React from 'react'
22
import styled from 'styled-components'
33

4-
export type StyledProps = {
4+
export type TermProps = {
55
index?: number
66
margin?: string
77
border_left?: boolean
88
}
99

1010
type TermsServiceType = { header?: string; text: string; icon: string }
1111

12-
type OurServiceType = { first?: TermsServiceType; other?: TermsServiceType[] }
12+
type OurServiceType = {
13+
first?: TermsServiceType
14+
other?: TermsServiceType[]
15+
}
16+
17+
type OurTermsType = {
18+
first_column?: TermsServiceType[]
19+
second_column?: TermsServiceType[]
20+
}
1321

14-
type OurTermsType = { first_column?: TermsServiceType[]; second_column?: TermsServiceType[] }
22+
export type PBType = {
23+
our_terms?: OurTermsType
24+
}
1525

1626
type DataType = {
1727
data?: {
@@ -41,7 +51,7 @@ const TermText = styled.div`
4151
text-align: left;
4252
`
4353

44-
const Term = styled.div<StyledProps>`
54+
const Term = styled.div<TermProps>`
4555
display: flex;
4656
align-items: center;
4757
width: 170px;

src/common/components/layout/layout.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import Footer from './footer'
33
import Header from './header'
44

55
type LayoutType = {
6-
children: React.ReactChild
6+
children: React.ReactChild | React.ReactChild[]
77
}
88

99
const Layout = ({ children }: LayoutType) => {

src/images/common/careers/index.ts

+6
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
import Sinbad0 from './sinbad-0.png'
2+
import Sinbad1 from './sinbad-1.png'
3+
import Sinbad2 from './sinbad-2.png'
4+
import Sinbad3 from './sinbad-2.png'
5+
6+
export { Sinbad0, Sinbad1, Sinbad2, Sinbad3 }

src/pages/careers/hiring-process.tsx

+8-1
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,14 @@ const CardInfo = styled.div`
4242
flex-direction: column;
4343
`
4444

45-
const data = [
45+
type DataType = {
46+
number: number
47+
icon: string
48+
header: string
49+
text: string
50+
}
51+
52+
const data: DataType[] = [
4653
{
4754
number: 1,
4855
icon: Random,

src/pages/careers/index.tsx

+5-7
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,11 @@ import JoinUs from 'pages/join-us'
99
const Careers = () => {
1010
return (
1111
<Layout>
12-
<>
13-
<WorkWithUs />
14-
<PerksBenefits />
15-
<OpenPositions />
16-
<HiringProcess />
17-
<JoinUs active={3} />
18-
</>
12+
<WorkWithUs />
13+
<PerksBenefits />
14+
<OpenPositions />
15+
<HiringProcess />
16+
<JoinUs active={3} />
1917
</Layout>
2018
)
2119
}

src/pages/careers/open-positions.tsx

+13-2
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,10 @@ const HeaderContainer = styled.div`
2020
padding: 5px 0;
2121
`
2222

23+
const StyledHeader = styled(Header)`
24+
text-transform: uppercase;
25+
`
26+
2327
const Card = styled(Flex)`
2428
display: flex;
2529
justify-content: space-between;
@@ -41,9 +45,16 @@ const CardButton = styled(Link)`
4145
background-size: 15px 15px;
4246
background-position: right;
4347
text-decoration: none;
48+
text-transform: uppercase;
4449
`
50+
type PositionsType = {
51+
icon: string
52+
position: string
53+
text: string
54+
link: string
55+
}
4556

46-
const positions = [
57+
const positions: PositionsType[] = [
4758
{
4859
icon: Random,
4960
position: 'Information security',
@@ -86,7 +97,7 @@ const OpenPositions = () => {
8697
return (
8798
<OpenPositionsContainer>
8899
<HeaderContainer>
89-
<Header>OPEN POSITIONS</Header>
100+
<StyledHeader>OPEN POSITIONS</StyledHeader>
90101
<Header font_weight="normal">Make your passon in your career</Header>
91102
</HeaderContainer>
92103
<Flex width="1100px" wrap="wrap">

src/pages/careers/perks-benefits.tsx

+3-3
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import styled from 'styled-components'
33
import Random from 'images/svg/random.svg'
44
import Sinbad1 from 'images/common/careers/sinbad-1.png'
55
import { Text, Header, ImageContainer } from 'common/components/containers'
6-
import Table from 'common/components/containers/table'
6+
import Table, { PBType } from 'common/components/containers/table'
77

88
const PBContainer = styled.div`
99
display: flex;
@@ -32,7 +32,7 @@ const PBTable = styled.div`
3232
font-weight: bold;
3333
`
3434

35-
const data = {
35+
const pb_data: PBType = {
3636
our_terms: {
3737
first_column: [
3838
{
@@ -94,7 +94,7 @@ const PerksBenefits = () => {
9494
<ImageContainer src={Sinbad1} />
9595
<PBTable>
9696
Our Terms
97-
<Table data={data} />
97+
<Table data={pb_data} />
9898
</PBTable>
9999
</PBContainer>
100100
)

src/pages/home/welcome.tsx

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import Sinbad1 from 'images/common/carousel/sinbad-1.png'
77
import Sinbad2 from 'images/common/carousel/sinbad-2.png'
88
import Flex from 'common/components/containers/flex'
99
import { Text, Header, ImageContainer } from 'common/components/containers'
10-
import Table, { StyledProps } from 'common/components/containers/table'
10+
import Table, { TermProps } from 'common/components/containers/table'
1111

1212
const WelcomeContainer = styled.div`
1313
width: 100%;
@@ -34,7 +34,7 @@ const CardContainer = styled.div`
3434
padding-bottom: 25px;
3535
`
3636

37-
const Card = styled(Flex)<StyledProps>`
37+
const Card = styled(Flex)<TermProps>`
3838
max-width: 324px;
3939
height: 290px;
4040
padding: 32px 44px 32px 32px;

src/pages/index.tsx

+4-6
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,10 @@ const Home = () => {
1616

1717
return (
1818
<Layout>
19-
<>
20-
<Carousel active={active} setActive={setActive} />
21-
<Welcome active={active} />
22-
<TechStack />
23-
<JoinUs active={active} />
24-
</>
19+
<Carousel active={active} setActive={setActive} />
20+
<Welcome active={active} />
21+
<TechStack />
22+
<JoinUs active={active} />
2523
</Layout>
2624
)
2725
}

src/pages/join-us.tsx

+1-4
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
11
import React from 'react'
22
import styled from 'styled-components'
33
import { IconProps } from './index'
4-
import Sinbad0 from 'images/common/carousel/sinbad-0.png'
5-
import Sinbad1 from 'images/common/carousel/sinbad-1.png'
6-
import Sinbad2 from 'images/common/carousel/sinbad-2.png'
7-
import Sinbad3 from 'images/common/careers/sinbad-2.png'
4+
import { Sinbad0, Sinbad1, Sinbad2, Sinbad3 } from 'images/common/careers'
85
import { SeeOurPositions, Header, ImageContainer, Text } from 'common/components/containers'
96
import Dashboard from 'images/common/dashboard.png'
107

src/pages/open-positions/index.tsx

+2-4
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,8 @@ import Layout from 'common/components/layout/layout'
66
const Positions = () => {
77
return (
88
<Layout>
9-
<>
10-
<OpenPositions />
11-
<JoinUs />
12-
</>
9+
<OpenPositions />
10+
<JoinUs />
1311
</Layout>
1412
)
1513
}

0 commit comments

Comments
 (0)