Skip to content

fix: [#1859] Only add the value of buttons to the form data if they are the submitter #1860

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix: [#1859] Only add the value of buttons to the form data if they are the submitter #1860

wants to merge 2 commits into from

Conversation

maxmil
Copy link
Contributor

@maxmil maxmil commented Jul 26, 2025

No description provided.

@maxmil maxmil requested a review from capricorn86 as a code owner July 26, 2025 09:23
@maxmil
fix: [capricorn86#1859] Fix failing test
10fa1a0 
Not sure why npm test did not detect that this test needed to be rerun after my changes.
@maxmil
Copy link
Contributor Author

maxmil commented Jul 26, 2025

Ah. I've just noticed #1759

Appologies for the duplication. This is another take on the same prolem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@capricorn86 capricorn86 Awaiting requested review from capricorn86 capricorn86 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@maxmil