Skip to content

Conversation

dnna
Copy link
Contributor

@dnna dnna commented Aug 25, 2018

No description provided.

@dnna
Copy link
Contributor Author

dnna commented Aug 25, 2018

This is fairly ready. It should close #128 except for the Laravel passport part. Someone more expert with Laravel should work with that :)

@andig andig merged commit a607582 into php-pm:master Aug 26, 2018
@andig
Copy link
Contributor

andig commented Aug 26, 2018

Someone more expert with Laravel should work with that :)

No need for that except for an entire Laravel testsuite. The JSON case you've implemented should already have caught #123 if it had been in place then.

Much appreciate, thank you!!!

@dnna dnna deleted the tests branch August 26, 2018 13:58
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.

2 participants