Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Added an option to add request data from controller #154

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

Merged
merged 1 commit into from
Jun 17, 2022

Conversation

bhekor
Copy link
Contributor

@bhekor bhekor commented Feb 1, 2022

I added an option for users (developers) to provide the data (request data) from the controller instead of a form (i.e you don't have to submit the data at least not all from a form). This adds some levels of security to the data and also ability to provide default values.

I added an option for user to provide the data (request data) from controller instead of a form. This adds some levels of security to the data and also ability to provide default values.
@bhekor bhekor changed the title Add option to add request data from controller Added an option to add request data from controller Feb 1, 2022
@sodmond
Copy link

sodmond commented Jun 17, 2022

Why has this pull request yet to be merged? Its actually what I need right now. I don't want to be manipulating the core files in the vendor folder which I don't think is advisable.

Is there other round I could use your solution?

@unicodeveloper unicodeveloper merged commit 00b00fe into unicodeveloper:master Jun 17, 2022
@sodmond
Copy link

sodmond commented Jun 17, 2022

@unicodeveloper thanks for the quick response. I just had to pull from dev-master to get the latest commit.

@raphyabak
Copy link

Please can you sync it with the packagist file. This update is still missing upon installation of the package. Thank you

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.

4 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.