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

Data Transformers are ignored on form processing #31

Open
parktrip opened this issue Jul 15, 2024 · 7 comments
Open

Data Transformers are ignored on form processing #31

parktrip opened this issue Jul 15, 2024 · 7 comments

Comments

@parktrip
Copy link

Dear,

first thank you for this very nice bundle, it works perfectly.
I just have a problem with Data Transformer, they are just completely ignored!
I think the issue was reported in #17 but no solution was provided.

If I try to add the Model Transformer to a dependent field, it just says the field does not exist.
If I try to attach it in the dependent field callback it doesn't work.

Is there any way to have model transformers working with dependent fields ?

Thank you in advance

@Hricer
Copy link

Hricer commented Oct 2, 2024

the same problem

@j0r1s
Copy link

j0r1s commented Nov 4, 2024

I have provided a solution in this closed ticket

@bocharsky-bw
Copy link
Member

Thanks for sharing the solution @j0r1s !

@parktrip @Hricer could you guys confirm it works for you too? If so, we could close this issue. Also, anyone is interested in creating a PR to improve the docs? We could mention the working example there if it will help others.

@j0r1s
Copy link

j0r1s commented Nov 4, 2024

If this approach is confirmed as the best solution, I’d be happy to work on a PR to improve the documentation.

While creating a custom form type, even for a simple data transformer, may seem a bit cumbersome, it’s likely the cleanest approach in the end.

@sadikoff
Copy link

Hey @Hricer @parktrip a gentle ping are you still interested in fixing it, or that's already fixed for you?

@Hricer
Copy link

Hricer commented Nov 25, 2024

@j0r1s It's a good solution. Thanks for sharing. It works in my case.

@parktrip
Copy link
Author

Hey @Hricer @parktrip a gentle ping are you still interested in fixing it, or that's already fixed for you?

sorry i didn't have the time to check the solution

@parktrip parktrip closed this as not planned Won't fix, can't repro, duplicate, stale Nov 25, 2024
@parktrip parktrip reopened this Nov 25, 2024
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

No branches or pull requests

5 participants