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

[react] store transform callback in useRef #1171

Closed
wants to merge 17 commits into from

Conversation

mrmonat
Copy link

@mrmonat mrmonat commented May 5, 2022

This should fix #1131

I think because the old implementation did not persist the callback between renders, the transform method of the form helper had no effect (was reset to default (data) => data callback) when called like shown in the example in issue #1131

@mrmonat mrmonat changed the title [react] use ref for storing the transform callback [react] store transform callback in useRef May 5, 2022
@mxmtsk
Copy link

mxmtsk commented May 24, 2022

I ran into the same issue and can confirm that this fixes the issue.

@ashlimx
Copy link

ashlimx commented Nov 8, 2022

Can we please put priority on this pull request? It's not great the documented transform feature just 100% does not work at all for this long with an active fix available

@mahangm
Copy link

mahangm commented Dec 9, 2022

Is there no update on this PR?

@Jamesking56
Copy link

This looks like it needs rebasing as it has a bunch of other unrelated changes in it

@ghost
Copy link

ghost commented Feb 25, 2023

This is a huge update! Can confirm this is working as expected. How can we get this merged? These are the only changes which are needed:

53b40db

@mrmonat
Copy link
Author

mrmonat commented Jun 2, 2023

#1491 is newer.

@mrmonat mrmonat closed this Jun 2, 2023
@mrmonat mrmonat deleted the patch-1 branch June 2, 2023 14:15
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.

FormHelper transform not working
9 participants