Results 1104 comments of Anders Jenbo

it reminds me of the old days (when i did frontend stuff too) when you had to "return false" on any link that did scripting rather then actually link somewhere....

Thanks, I did a bit extra refactoring to change the code to use Point rather then int pairs, this also removed some redundant code.

... was it not ready for merge?

With https://github.com/larastan/larastan/pull/1861 merged this PR can now be closed as it's a duplicate

I don't think Bladestan knows how to analyze `Illuminate\Mail\Mailables\Content` so this will just result in the template being ignored.

Here is what you can do instead and still have it analyzed: ```php return $this->to('[email protected]') ->subject('[Payin] n°' . $this->ressourceId) ->html(view('email.toUser.payinFailed') ->with([ 'payinId' => $this->ressourceId, ])); ``` That's just 6 extra...

Thanks this looks good, but please revert the PHPUnit requirement changes

> > Thanks this looks good, but please revert the PHPUnit requirement changes > > Done - for me, this means I can no longer run either "composer install" on...

It looks like it could happen in 1.6.0