Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants