New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make radio buttons in interest form work #3695
Make radio buttons in interest form work #3695
Conversation
They did not work due to `type="radio` being passed, since this causes, `react-final-form` to include a `checked` prop as well. There was also a seperate bug with the "office in Trondheim" question not working, since you the object keys were booleans.
@@ -58,7 +58,7 @@ const mapStateToProps = (state, props) => { | |||
checked: false, | |||
})), | |||
participantRange: null, | |||
companyTypes: null, | |||
companyType: null, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was misspelled, but had no affect on anything. Not sure why it is even needed.
@@ -660,7 +659,7 @@ const CompanyInterestPage = (props: Props) => { | |||
|
|||
return ( | |||
<Content> | |||
<Helmet title="Bedriftsinteresse" /> | |||
<Helmet title={isEnglish ? 'Company interest' : 'Bedriftsinteresse'} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unrelated, but just nice to have.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 💯
Description
They did not work due to
type="radio
being passed, since this causes,react-final-form
to include achecked
prop as well.There was also a seperate bug with the "office in Trondheim" question not working, since the object keys were booleans.
Testing
I created and edited interest forms. I especially tested the "office in trondheim" question.