add morph type to returned models

[crissi]

when you retrieve the morphed models in your query resolver the type is missing so don't know what type it is

Are we going to continue to support 5.5? can see the tests are failing for that version

[mfn]

Are we going to continue to support 5.5? can see the tests are failing for that version

Thanks for asking, I haven't yet disclosed my thoughts on this.

Basically I'm following https://laravel.com/docs/master/releases

That's why I also removed tests for 5.6 and 5.7 without much discussion via #488

I basically took the easy way out: I just stick to the LTS table. Once a version is beyond security fixes it can be removed on first sign of troubles.

[crissi]

Are we going to continue to support 5.5? can see the tests are failing for that version

Thanks for asking, I haven't yet disclosed my thoughts on this.

Basically I'm following https://laravel.com/docs/master/releases

That's why I also removed tests for 5.6 and 5.7 without much discussion via #488

I basically took the easy way out: I just stick to the LTS table. Once a version is beyond security fixes it can be removed on first sign of troubles.

Ok will make a fix for laravel 5.5

[crissi]

how do I actually run this for Laravel 5.5 locally? I tried removing all other versions from illuminate/contracts, illuminate/support in the composer.json, but gets alot of composer errors.

[mfn]

I believe from .travis.yml, these lines from the install section are relevant:

composer require "illuminate/support:${LARAVEL}" --no-interaction --no-update
composer install --prefer-dist --no-interaction --no-suggest

[crissi]

Thanks, will try Hent Outlook til iOS<https://aka.ms/o0ukef>

…

________________________________ Fra: Markus Podar <[email protected]> Sendt: Monday, October 28, 2019 12:24:53 PM Til: rebing/graphql-laravel <[email protected]> Cc: Christian <[email protected]>; Author <[email protected]> Emne: Re: [rebing/graphql-laravel] add morph type to returned models (#503) I believe from .travis.yml, these lines from the install section are relevant: composer require "illuminate/support:${LARAVEL}" --no-interaction --no-update composer install --prefer-dist --no-interaction --no-suggest — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#503?email_source=notifications&email_token=AAXPDYQMTHTPLYWM3G3G73TQQ3DYLA5CNFSM4JFAWYQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECMRM2I#issuecomment-546903657>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAXPDYSTE5775SCJAXGVGZDQQ3DYLANCNFSM4JFAWYQA>.

[mfn]

Thank you very much 🤗