graphql-loader

graphql-loader copied to clipboard

Use of an @import directive

3

There are 2 proposed ways of importing graphql files into one another and the 1st one is covered by this loader. The other one is through the use of an...

emahuni

Double Pound sign ## for comment that this loader doesn't parse

2

Double pound should be a real comment even if it contains a valid or invalid import directive. eg: ```graphql #import "./file.graphql"

emahuni

External fragments are not pulled into the query document using the loader

6

import doesn't seem to contain the fragment specified in # import statement and doesn't handle single quoted paths as well: contact.fragment.graphql: ```graphql fragment fullContactFields on Contact { addresses { line1...

emahuni

It would be nice to support multiple operations (queries/mutations) per module, which get exported as named exports for instance. The loader included in the graphql-tag package supports this. How difficult...

simoneb

Warning during installation: "The 'apollo-codegen' command has been replaced...

1

I'm seeing this warning when doing a fresh yarn install of a project containing [email protected], the latest version at time of writing: > The 'apollo-codegen' command has been replaced with...

aneilbaboo

ts-loader is both a devDependency and peerDependency at incompatible version ranges

4

In `devDependencies`, there is `"ts-loader": "3.x",` but in `peerDependencies` there's `"ts-loader": "^4.4.1"` I'm not sure why `ts-loader` is a peer dependency; seems like it should only be a development dependency?

ThisIsMissEm

stephen

fix this: https://github.com/samsarahq/graphql-loader/blob/master/src/loader.ts#L194

stephen

Option to automatically add ID field to queries

1

In order for automatic cache updates to work correctly, Apollo (by default) requires that the id and __typename fields are provided in queries. It is very easy to forget to...

chrisdevereux

stephen