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

Add custom binding functions to CrudFormFactory #128

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

miku76
Copy link

@miku76 miku76 commented Dec 8, 2023

With the added functions it is possible to define your own ValueProvider (getter) and a setter for a property.
This allows you to add a form field with your own getter/setter methods, analogous to the additional columns in the grid.

@alejandro-du
Copy link
Owner

Nice! Thanks a lot for this PR. Sorry for the delay response. I quickly checked the changes and there are many that seem to be formatting of code which makes it hard to understand the actual change in functionality. Would you mind removing any changes that are related to formatting so I can test and merge your contribution without having to cherry-pick changes and potentially erasing your name from the commit history? I want to give you full credit for the code changes. Thanks again!

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.

2 participants