Skip to content

[LiveComponent] Fix ComponentWithFormTrait not working in batch actions #2553

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

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

smnandre
Copy link
Member

@smnandre smnandre commented Feb 6, 2025

Q A
Bug fix? yes
New feature? no
Issues Fix #1509
License MIT

Fix #1509

Thank you @jpvdw86 for your work on this! 👏

Some parts of your PR would have introduced changes affecting all LiveComponent instances, including those not using ComponentWithFormTrait. To keep the impact as minimal as possible, I’ve opted for a more targeted fix.

But you did all the hard work here—much appreciated! 🚀

Fix symfony#1509

Thank you @jpvdw86 for your work on this!

Some parts of your PR would have introduced changes affecting all LiveComponent instances, including those not using ComponentWithFormTrait. To keep the impact as minimal as possible, I’ve opted for a more targeted fix.

You did all the hard work here—much appreciated! 🚀
@carsonbot carsonbot added Status: Reviewed Has been reviewed by a maintainer and removed Status: Needs Review Needs to be reviewed labels Feb 7, 2025
@smnandre smnandre merged commit feb0256 into symfony:2.x Feb 7, 2025
59 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Bug Fix LiveComponent Status: Reviewed Has been reviewed by a maintainer
Projects
None yet
3 participants