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

Update web.php #4403

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@flick36

flick36 commented Sep 2, 2017

Since we now have the new view route helper we can use it on the web.php routes file isn't it? cleaning the code a little bit

Update web.php
Since we now have the new view route helper we can use it on the web.php routes file isn't it? cleaning the code a little bit

@flick36 flick36 closed this Sep 2, 2017

@joshmanders

This comment has been minimized.

Contributor

joshmanders commented Sep 2, 2017

seriously people, look at previous PR's both closed and open before you submit the same PR 60 times.

@antonkomarev

This comment has been minimized.

antonkomarev commented Sep 2, 2017

@joshmanders I hope after 1000th PR #4401 will be accepted to stop this.

@flick36 flick36 deleted the flick36:patch-1 branch Sep 2, 2017

@flick36

This comment has been minimized.

flick36 commented Sep 2, 2017

@joshmanders u mad?

@brunogaspar

This comment has been minimized.

Contributor

brunogaspar commented Sep 2, 2017

@flick36 I don't believe he's mad, but i totally see his point, it's very frustrating to see the same PR being done every, single, day when there's search functionality to determine if a PR was done or not for this change (or any other change).

Cheers 🍻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment