-
Notifications
You must be signed in to change notification settings - Fork 0
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
253 feature create form instance flow #297
base: main
Are you sure you want to change the base?
Conversation
…53-feature-create-form-instance-flow
…b.com/sandboxnu/mfa-form-automator into 253-feature-create-form-instance-flow
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Looking good so far– could we merge in changes from main soon? There are a lot of type changes that have to be introduced to this PR from main that will help complete things
disabled={disabled} | ||
review={review} | ||
/> | ||
{isFormTemplate ? ( |
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 can consolidate this conditional to:
<FormButtons
isFormTemplate={isFormTemplate}
heading=`Create Form ${isFormTemplate ? "Template" : "Instance" }`
deleteFunction={deleteFunction}
submitLink={submitLink}
backLink={backLink}
disabled={disabled}
review={review}
/>```
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.
@kaiyangzheng also says that we need to add a home button at the end
Description
Created flow pages for when creating a form instance.
Motivation and Context
Allows users to create form instances in a similar flow as form templates and assign users to field groups
How has this been tested?
Created form instances from pre-existing form templates.
Screenshots (if appropriate):
Types of changes
Also added signaturePosition fields to formInstanceContext so assignees can be saved and shown on review page.
Checklist:
What is the code style and how do i run tests?