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.

Sign up for GitHub

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

Jump to bottom

[Insight for Laravel] Verify that there is no usages of requests outside FormRequests or Controllers #13

Closed
nunomaduro opened this issue Apr 9, 2019 · 2 comments
Closed

[Insight for Laravel] Verify that there is no usages of requests outside FormRequests or Controllers #13

nunomaduro opened this issue Apr 9, 2019 · 2 comments
Labels
enhancement New feature or request

Comments

@nunomaduro
Copy link
Owner

nunomaduro commented Apr 9, 2019 •
edited
Loading

To solve this issue, you should create a pull request adding the Insight described on the title.
@nunomaduro nunomaduro added the enhancement New feature or request label May 12, 2019
@ijpatricio
Copy link

Maybe leaving the classes where the the request can be used as configurable.

Think for example, DataTransferObjects. In https://github.com/spatie/data-transfer-object , a fromRequest constructor is adopted.

There might be more cases, so leaving calss / parent class as configurable might be a good solution.

@jrmadsen67 jrmadsen67 mentioned this issue Feb 2, 2020
Closed
@nunomaduro
Copy link
Owner Author

No plans to work on this at this time.

@devkishanu devkishanu mentioned this issue Feb 15, 2024
Open
@hdcore hdcore mentioned this issue Oct 12, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nunomaduro @ijpatricio