fix(api): Add @types/graphql dependency #2369
Conversation
- Add type guard for DocumentNode - Refactor `getGraphqlOperationType` Fixes aws-amplify#2365 Fixes aws-amplify#2362
ghost
assigned 
|
Will this cause issues with consumers who aren't including 'esnext' in their tsconfig.json |
|
@nihakue In fact it's generating new issues like described here: #2365 @manueliglesias Are you working on it? When should we expect the merge? |
|
Any news on this PR? |
| @@ -53,6 +53,7 @@ | |||
| "@aws-amplify/auth": "^1.2.13", | |||
| "@aws-amplify/cache": "^1.0.20", | |||
| "@aws-amplify/core": "^1.0.20", | |||
| "@types/graphql": "0.13.0", | |||
There was a problem hiding this comment.
Move these types to devDependencies.
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This issue has been automatically closed because of inactivity. Please open a new issue if are still encountering problems. |
|
This pull request has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs. Looking for a help forum? We recommend joining the Amplify Community Discord server |
getGraphqlOperationTypeFixes #2365
Fixes #2362
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.