Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Upgrade Dependencies #20

Closed
10 tasks done
cerinoligutom opened this issue Nov 24, 2021 · 2 comments
Closed
10 tasks done

Upgrade Dependencies #20

cerinoligutom opened this issue Nov 24, 2021 · 2 comments
Assignees
@cerinoligutom
Labels
dependencies Pull requests that update a dependency file

Comments

@cerinoligutom
Copy link
Owner

cerinoligutom commented Nov 24, 2021 •
edited
Loading

This issue is mainly to track dependencies that needs updating as of opening the issue.

  • knex@latest from 0.95.4
  • objection@3 from 2.2.14
  • supertokens-node@8 from 7.0.0
  • eslint and associated plugins
  • typescript@4.5 from 4.3.5
  • Docker Image node:14-alpine to node:16-alpine
  • graphql@16 from 15.7.2
  • graphql-subscriptions@2 from 1.2.1
  • graphql-codegen and associated plugins
  • graphql-ws with backwards-compat to subscriptions-transport-ws
@cerinoligutom cerinoligutom added the dependencies Pull requests that update a dependency file label Nov 24, 2021
@cerinoligutom cerinoligutom self-assigned this Nov 24, 2021
@cerinoligutom
Copy link
Owner Author

cerinoligutom commented Nov 24, 2021 •
edited
Loading

Tried updating graphql-codegen but encountered the same issue as this. Turns out there's a deeper issue.

Relevant links:
#232 and #240 from apollographql/graphql-subscriptions

Might consider using patch-package for this issue.

Edit: Used patch-package to solve the issue.

@cerinoligutom
Copy link
Owner Author

cerinoligutom commented Nov 24, 2021 •
edited
Loading

Tried migrating to graphql-ws and this unfortunately breaks the GraphQL Playground as it's using a different protocol spec.

Relevant links:

Edit: Figured it out. Pain.

cerinoligutom added a commit that referenced this issue Nov 24, 2021
@cerinoligutom
build(deps): update graphql-codegen
8c01f28 
See link below for the source of pain
#20 (comment)
TopDev113 added a commit to TopDev113/GraphQL-Starter that referenced this issue Jun 20, 2024
@TopDev113
build(deps): update graphql-codegen
4bfcec8 
See link below for the source of pain
cerinoligutom/GraphQL-Starter#20 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cerinoligutom cerinoligutom

Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cerinoligutom