Skip to content

Migrate to Apollo v5#119

Merged
pawiecz merged 14 commits into
mainfrom
migrate-to-apollo-v5
Apr 14, 2026
Merged

Migrate to Apollo v5#119
pawiecz merged 14 commits into
mainfrom
migrate-to-apollo-v5

Conversation

@pawiecz
Copy link
Copy Markdown
Contributor

@pawiecz pawiecz commented Mar 17, 2026

Context & Requests for Reviewers

Fixes: #67

Tests

  • Server tests passed locally
  • Generated GraphQL contains no changes
  • Server and Client are operational in local environment

Base automatically changed from migrate-to-vite to main March 24, 2026 02:32
@pawiecz
Copy link
Copy Markdown
Contributor Author

pawiecz commented Mar 24, 2026

This PR suffers from the same issue as #74 (missing Vite configuration in Docker build context).

I'll rebase it on current main and undraft it.

@pawiecz pawiecz force-pushed the migrate-to-apollo-v5 branch from 739d331 to 27505f4 Compare March 31, 2026 14:56
@pawiecz pawiecz marked this pull request as ready for review March 31, 2026 15:01
Copy link
Copy Markdown
Contributor

@vinaysrao1 vinaysrao1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor requests only. this is a clean and methodically done PR. Well done.

Comment thread server/api.ts Outdated
Comment thread server/package.json Outdated
Comment thread server/api.ts Outdated
Comment thread server/graphql/modules/apiKey.ts Outdated
Comment thread server/graphql/modules/apiKey.ts Outdated
@juanmrad juanmrad requested review from juanmrad and vinaysrao1 April 14, 2026 01:25
Copy link
Copy Markdown
Contributor

@vinaysrao1 vinaysrao1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM

@pawiecz pawiecz merged commit ef1b5bf into main Apr 14, 2026
11 checks passed
@pawiecz pawiecz deleted the migrate-to-apollo-v5 branch April 14, 2026 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate apollo-* server dependencies that reached EOL

3 participants