-
Notifications
You must be signed in to change notification settings - Fork 1
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
STATFLO-1093 #16
STATFLO-1093 #16
Conversation
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.
Looks awesome 👏 Can't wait to see this live!
"private": true, | ||
"dependencies": { | ||
"@faker-js/faker": "^8.0.2", | ||
"@statflo/ui": "^0.0.14", |
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.
cool
|
||
import { useWidgetContext } from "../providers/WidgetProvider"; | ||
|
||
const WidgetSchema = Yup.object().shape({ |
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 haven't heard of yup. Is it a form builder?
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.
Or validation?
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.
It's a schema builder, which I am using for validating the form. It's what is recommended by Formik
which is the form package that I am using.
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.
Cool. I'm a fan of form builders, I was wondering if there was any motivation to use on our react projects.
widget?: Widget; | ||
}; | ||
|
||
const WidgetForm = ({ isEdit, widget }: WidgetFormProps) => { |
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.
Here you simulate the widget admin? I know you said you were attempting this, very nice.
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.
Yeah! It's basically a dumbed down version of widget admin so that it simulates what would actually be going on in the application.
No description provided.