Tim Kelty
Tim Kelty
> I'll browse through the https://github.com/craftcms/cms issue list and see if moving previews away from POST data is on the horizon and if not I'll open an issue to ask...
@pauloelias Indeed I am still running my branch in production, unfortunately. Here's my changes: https://github.com/markhuot/craftql/compare/master...timkelty:dlr2018 I kept it up to date with changes for a while, but haven't recently. I...
> Yup, after I get this lazy loading in place I'd like to get Apollo Engine supported so we can use their per-key caching layers. 🥇
@pauloelias my setup is dockerized too, so I'll try this today and see what I get
Hmmm for me on `3.0.40` (and on `3.1`) on `markhuot/[email protected]` I'm getting the same as @pauloelias : ``` $ docker-compose --file='./ops/docker/dlr_craftcms/docker-compose.yml' exec app bin/craft craftql/tools/print-schema Exception 'GraphQL\Error\InvariantViolation' with message 'Each...
@pauloelias if you're still considering using my branch to get you by in the interim, I rebased it off markhuot/master. Nothing _appears_ to have conflicted or broken: https://github.com/timkelty/craftql/tree/response-caching
> Phew, that's definitely an edge case. HOW DARE YOU insult convoluted my content model! 😁 But in all seriousness…yeah I think i'd prefer something closer to option 1…I'm wondering...
A more foreseeable clash could happen in the near future…consider this: - Craft 3.next gives Categories "Types", like entries (which seems pretty likely to happen) - Someone has a Section...
> I like {fieldHandle}{blockHandle}Matrix ideally, but that'd be a breaking change so I'm worried about introducing it so late in the game. Adding it in only when needed feels safer...
Sounds good! 👍 In all honesty the error let me know how effed up things were in our schema, and I just went and changed it 😆