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

[US-234] Update form example to add a submit button #239

Merged
merged 5 commits into from
Jan 23, 2024

Conversation

3ace
Copy link
Contributor

@3ace 3ace commented Jan 7, 2024

No description provided.

@3ace 3ace requested review from gunnsth and sampila January 7, 2024 11:03
@3ace 3ace force-pushed the us-234-form-submission-example branch from 46c4c73 to 5137e94 Compare January 7, 2024 15:18
Copy link
Contributor

@sampila sampila left a comment

Choose a reason for hiding this comment

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

Probably best to separate the example? like form_external_submit.go?

@3ace
Copy link
Contributor Author

3ace commented Jan 8, 2024

@sampila is there a particular reason for that? Since I think the submit button is related with form. But if there's a good reason for it then I'm okay to separate the example.

@gunnsth
Copy link
Contributor

gunnsth commented Jan 9, 2024

Yeah might be good to separate. The other one is just focused on adding some basic things to the form but then this one adds the submit functionality which I am not sure if is very commonly used.
Then the use case is also clear and easier to find.

@3ace
Copy link
Contributor Author

3ace commented Jan 9, 2024

understood, I'll update the PR then. Thanks for the input

@3ace 3ace marked this pull request as draft January 9, 2024 17:20
@3ace 3ace marked this pull request as ready for review January 23, 2024 04:26
@3ace
Copy link
Contributor Author

3ace commented Jan 23, 2024

@sampila @gunnsth I've updated this PR to use the new submit functionality introduced in unipdf v3.54

@3ace 3ace requested a review from sampila January 23, 2024 04:37
Copy link
Contributor

@sampila sampila left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@gunnsth gunnsth merged commit a854a94 into unidoc:master Jan 23, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants