Generate optional TypeScript fields when @include or @skip directive is used

[mwilliamson]

This makes generated TypeScript fields optional if the @include directive is used. Fixes #888.

[apollo-cla]

@mwilliamson: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

[OzzieOrca]

Does this support @skip as well?

[mwilliamson]

Does this support @skip as well?

Yes, @skip should also be supported. I've added a test case to cover this.

I've also signed the CLA.

[MargaretKrutikova]

Very much appreciated, @mwilliamson! Just stumbled upon this issue at work, would be great to have it merged 🙂

[hlehmann]

any news on this ?

[chaffeqa]

If it helps, we are using this PR (via https://www.npmjs.com/package/patch-package) and its working great! Thx @mwilliamson

[dobesv]

I wonder if rebasing on latest master will clear up that build error.

[pelhage]

We ran into a bug caused by this issue. Would be nice to see this PR introduced!

[vidmantasa]

Any news on this?

[audriuss]

Please, merge the PR :) Please

[jaminhaber]

Can this be merged? It would come in very handy for me on a project

[samageloff]

Really looking forward to this!

[mwilliamson]

I'm closing this since it seems like it's not going to be merged. Regardless, when I was still working on a codebase using GraphQL, we switched to GraphQL Code Generator.