Use graphiql option variables if query variables are not supplied

[alanchristensen]

TODO:

• Update CHANGELOG.md with your change (include reference to issue & this PR)
• Make sure all of the significant new logic is covered by tests
• Rebase your changes on master so that they can be merged easily
• Make sure all tests and linter rules pass

Closes #193

[apollo-cla]

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

[DxCx]

Looks good in general, just one question, why did you remove the typings there?

[DxCx]

Why removing the typings...?

[alanchristensen]

Hi DxCx,
createOptions can be either a HapiPluginOptions or GraphiQLPluginOptions now. My options were to either create a new type just for this test function that contained a graphqlOptions and graphiqlOptions or remove the type.

Is there another option that I'm missing? My typescript is pretty weak.

[DxCx]

createOptions: (HapiPluginOptions | GraphiQLPluginOptions) should work ;)

[alanchristensen]

Aha, that worked. I had to cast and check for the presence of .graphqlOptions and .graphiqlOptions further below.

[DxCx]

Thanks! :)

[DxCx]

@helfer lets merge this as well as soon as @alanchristensen is rebasing the code upon latest :)

[DxCx]

0.5.1 branch is merged :)
@alanchristensen can you rebase? or i'll?

[alanchristensen]

@DxCx I've rebased it.

[DxCx]

thanks @alanchristensen :)
@helfer can we merge?

[helfer]

Thanks @alanchristensen ! 🙂