Skip to content
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

feat: input file #677

Merged
merged 13 commits into from
Dec 19, 2023
Merged

feat: input file #677

merged 13 commits into from
Dec 19, 2023

Conversation

sdelamo
Copy link
Contributor

@sdelamo sdelamo commented Dec 18, 2023

This PR adds another FormElement to handle input="file". If you have a filed of type CompletedFileUpload is considered automatically of type InputFileFormElement.

Additionally, this PR adds the FormElementResolver API to allow the FieldsetGenerator API to be extended by users.

@sdelamo sdelamo linked an issue Dec 18, 2023 that may be closed by this pull request
@sdelamo sdelamo changed the title Input 675 feat: input file Dec 18, 2023
@sdelamo sdelamo added the type: improvement A minor improvement to an existing feature label Dec 18, 2023
@sdelamo sdelamo requested a review from timyates December 18, 2023 12:34
@sdelamo sdelamo marked this pull request as ready for review December 18, 2023 12:34
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These can't be avoided, but they're impossible to review 😿

sdelamo and others added 2 commits December 18, 2023 22:30
…mentresolvers/CompositeFormElementResolver.java

Co-authored-by: Tim Yates <tim.yates@gmail.com>
@sdelamo sdelamo requested a review from timyates December 18, 2023 21:33
Copy link

@sdelamo sdelamo merged commit 16cdae4 into master Dec 19, 2023
25 checks passed
@sdelamo sdelamo deleted the input-675 branch December 19, 2023 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: improvement A minor improvement to an existing feature
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Support InputFileFormElement
2 participants