-
Notifications
You must be signed in to change notification settings - Fork 46
Gblosser/5931 hide save button button only #7005
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
Gblosser/5931 hide save button button only #7005
Conversation
doug-s-nava
left a comment
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 good with the note that there is stuff in here that will bear fruit in the follow up ticket #7067
| console.error(`Unknown widget type: ${type}`, definition); | ||
| throw new Error(`Unknown widget type: ${type}`); | ||
| } | ||
| if (props.isFormLocked) { |
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.
note that isFormLocked is not being passed, so this won't do anything yet
| required?: boolean; | ||
| disabled?: boolean; | ||
| readonly?: boolean; | ||
| readOnly?: boolean; |
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.
note that readOnly is not being passed yet, so implementation of this will not be used, but may later be depending on the requirements for #7067
Summary
Work for #5931
Changes proposed
When the form is not in progress, the Save button is hidden
This PR includes work that will be implemented in a future PR for #5931
Context for reviewers
This PR is part of larger work on the #5931 ticket, but only features the hidden save button change. The readonly changes are also present in this PR.
Validation steps
After submitting an application, the Save button should not be visible on forms.