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

feature: add BindForm function #3479

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

dozheiny
Copy link

Adding the BindForm function to the context. This feature will make it much easier for developers to handle form data in their applications. The added flexibility will improve the user experience and make the development process more efficient.

Copy link
Contributor

@alirezaeftekhari alirezaeftekhari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I agree with you,
We Usually use the Bind method that dedicates the content-type and handles the functionality with that.
But it will be better if we have another method that handles forms explicitly.
It would be better if you also implement the ShouldBindForm method and its corresponding test.

@dozheiny
Copy link
Author

dozheiny commented Feb 2, 2023

Hi;
Sure, I will commit to the changes you asked for.

@dozheiny
Copy link
Author

dozheiny commented Feb 4, 2023

I made the changes you asked for.

@dozheiny
Copy link
Author

Would one of the maintainers be able to review this PR?

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.

None yet

2 participants