fix:TalComUser Application form acknowledgment and redirection #1850
Conversation
|
jeyanthi-sm
force-pushed
the
jeyanthi_sm/talComPageAck1470
branch
from
c6ce685 to
be2dabf
Compare
…chers/dev-launchers-platform into jeyanthi_sm/talComPageAck1470
| <SignUpForm /> | ||
| {/* <TalFormPage></TalFormPage> */} | ||
| <SignUpForm handleCloseModal={handleCloseModal} /> | ||
| {/* <TalCommForm></TalFormPage> */} |
There was a problem hiding this comment.
is this commented part required to keep?
There was a problem hiding this comment.
It was commented before and it was not part of this change.
| @@ -157,13 +158,6 @@ export default function TalCommForm({ handleCloseModal, position }: Props) { | |||
| onChange={handleSetCheckCheckbox} | |||
| required | |||
| /> | |||
| <Checkbox | |||
| label="I have read and agreed to the terms and conditions" | |||
There was a problem hiding this comment.
Why remove this checkbox? I think it's nice to have it.
Applying without agreeing to the terms and conditions might become problematic down the line.
There was a problem hiding this comment.
After the review, I added the Terms and Conditions checkbox and reverted the commented lines on the 'Second' page to their previous state.
…everted the commented lines on the 'Second' page to their previous state.
TalComUser Applicant form acknowledgment and redirection