Skip to content

Commit ec62f72

Browse files
authored
Merge pull request #714 from topcoder-platform/issues-708-to-712
2 parents 793163a + 954d22e commit ec62f72

File tree

5 files changed

+30
-16
lines changed

5 files changed

+30
-16
lines changed

src/apps/onboarding/src/components/FormField/styles.module.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ $green1: #137d60;
55
display: flex;
66
flex-direction: column;
77
justify-content: flex-start;
8-
margin-bottom: 18px;
8+
margin-bottom: 0;
99

1010
.form-field {
1111
border: 1px solid #b7b7b7;

src/apps/onboarding/src/components/modal-add-education/styles.module.scss

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,11 @@
1414
}
1515
}
1616

17+
:global(.modal-body) {
18+
padding-bottom: 0;
19+
margin-bottom: 0;
20+
}
21+
1722
hr {
1823
margin-top: 30px;
1924
}

src/apps/onboarding/src/components/modal-add-work/index.tsx

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -223,17 +223,19 @@ const ModalAddWork: FC<ModalAddWorkProps> = (props: ModalAddWorkProps) => {
223223
</FormField>
224224
</div>
225225
</div>
226-
<FormInputCheckboxMiddleware
227-
label='I am currently working in this role'
228-
checked={workInfo.currentlyWorking}
229-
inline
230-
onChange={(e: any) => {
231-
setWorkInfo({
232-
...workInfo,
233-
currentlyWorking: e.target.checked,
234-
})
235-
}}
236-
/>
226+
<div className='mt-16'>
227+
<FormInputCheckboxMiddleware
228+
label='I am currently working in this role'
229+
checked={workInfo.currentlyWorking}
230+
inline
231+
onChange={(e: any) => {
232+
setWorkInfo({
233+
...workInfo,
234+
currentlyWorking: e.target.checked,
235+
})
236+
}}
237+
/>
238+
</div>
237239
</div>
238240
</BaseModal>
239241
)

src/apps/onboarding/src/pages/educations/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ export const PageEducationsContent: FC<{
6060

6161
return (
6262
<div className={classNames('d-flex flex-column', styles.container)}>
63-
<h2>Add your education information</h2>
63+
<h2>Education</h2>
6464
<PageDivider />
6565

6666
<div className={classNames('d-flex flex-column align-items-start full-width mt-8', styles.blockContent)}>

src/apps/onboarding/src/pages/personalization/index.tsx

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import { FC, MutableRefObject, useEffect, useRef } from 'react'
88
import classNames from 'classnames'
99

1010
import { Button, PageDivider } from '~/libs/ui'
11+
import { EnvironmentConfig } from '~/config'
1112

1213
import { ProgressBar } from '../../components/progress-bar'
1314
import styles from './styles.module.scss'
@@ -54,7 +55,11 @@ const PagePersonalizationContent: FC<{
5455

5556
useEffect(() => {
5657
if (!loading && !shouldSavingData.current && !!shouldNavigateTo.current) {
57-
navigate(shouldNavigateTo.current)
58+
if (shouldNavigateTo.current.startsWith('../')) {
59+
navigate(shouldNavigateTo.current)
60+
} else {
61+
window.location.href = shouldNavigateTo.current
62+
}
5863
}
5964
/* eslint-disable react-hooks/exhaustive-deps */
6065
}, [loading])
@@ -131,9 +136,11 @@ const PagePersonalizationContent: FC<{
131136
iconToLeft
132137
onClick={() => {
133138
if (loading) {
134-
shouldNavigateTo.current = '../account-details'
139+
shouldNavigateTo.current
140+
= `${EnvironmentConfig.USER_PROFILE_URL}/${props.memberInfo?.handle}`
135141
} else {
136-
navigate('../account-details')
142+
window.location.href
143+
= `${EnvironmentConfig.USER_PROFILE_URL}/${props.memberInfo?.handle}`
137144
}
138145
}}
139146
>

0 commit comments

Comments
 (0)