-
Notifications
You must be signed in to change notification settings - Fork 96
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
Creating dynamic form using ConnectorProperty #60
Conversation
uidoyen
commented
Sep 8, 2020
- Created dynamic form using connector properties
- Upgraded Formik
- Added Switch Component
- Added Formik to nohoist
@@ -147,6 +147,7 @@ export const CreateConnectorPage: React.FunctionComponent = () => { | |||
propertyValues: Map<string, string>, | |||
category: PropertyCategory | |||
): void => { | |||
console.log(propertyValues) |
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.
please remove the console.log
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.
I'm good with your changes. console.log was giving a build error - pls remove that then you can merge it.
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.
one suggestion in the tslint.json file rest looks good to me.
ui/packages/ui/tslint.json
Outdated
@@ -7,6 +7,7 @@ | |||
}, | |||
"rules": { | |||
"jsx-no-lambda": false, | |||
"ordered-imports": false, |
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.
I think we could leave the ordered-import true, as vscode(lint plugins) itself provides a one-click fix for it.