8000 FormRequest->withValidator is not working as validateResolved is not getting called. by sudkumar · Pull Request #1654 · dingo/api · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
8000

FormRequest->withValidator is not working as validateResolved is not getting called. #1654

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

Closed
wants to merge 7 commits into from

Conversation

sudkumar
Copy link
@sudkumar sudkumar commented May 6, 2019

Fixes #1653

gazben added 2 commits May 7, 2019 12:43
Allow https route declaration with 'https' array key and value.
```
$router->group(['https' => true], function (Router $router) {
});
```
@sudkumar
Copy link
Author

@specialtactics, Can you please look into it ?

Allow https route declaration with 'https' array key and value
@specialtactics
Copy link
Member

Hey @sudkumar sorry mate, I've been very busy, I will look at this shortly!

@specialtactics specialtactics self-requested a review June 19, 2019 05:19
@sudkumar
Copy link
Author

Thank you. @specialtactics I used the PR's commit till now.

I did not find any test case that I should update. So please let me know where/how I should write one if needed.

FormRequest->withValidator is not working as validateResolved is not getting called.
@specialtactics
Copy link
Member

@sudkumar I don't know why, but github says there's conflicts, so pull master and update please. I've had a look, and everything looks fine.

The laravel service provider actually doesn't even have this, which is odd - I will go and look into that separately now as well.

@sudkumar
Copy link
Author

Alright. I'll update.

@sudkumar sudkumar closed this Jun 20, 2019
@sudkumar
Copy link
Author
sudkumar commented Jun 20, 2019

Hi @specialtactics . I swear I did not close this. It says the commits have been merged!! I did not get it :/

@specialtactics
Copy link
Member

Hey @sudkumar I think you've done something funny while pulling in master 😛

Maybe just sync your repo and make a fresh PR ? Alternatively I could just make the change for you.

Uh oh!

There was an error while loading. Please reload this page.

@specialtactics
Copy link
Member

Actually @sudkumar it seems like it was in fact already merged. Very weird github says closed instead of merged!

@sudkumar
Copy link
Author

I only took pull from dingo/api#master and pushed it back. No idea. =D

But it is merged and that's what we need. Thank you. :)

@specialtactics
Copy link
Member

I added the same to the LaravelServiceProvider now:
#1666

Thanks for your contribution !

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.

FormRequest->withValidator is never getting called
3 participants
0