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.

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

Use Apollo platform API #1074

Merged
merged 4 commits into from
May 2, 2022
Merged

Use Apollo platform API #1074

merged 4 commits into from
May 2, 2022

Conversation

pcarrier
Copy link

@pcarrier pcarrier commented Apr 9, 2022

Migrate to the endpoint and schema we want publically documented and supported.

  • Change the default endpoint URL;
  • Migrate queries to the improved schema (renames like service → graph, and shape shifting like relocating fields from graph to variant rather than take a variant name argument);
  • Minimally restructure the client code around the new response shapes;
  • Quick fix for the git_context test relying on the repo's origin remote being the apollographql org, which eg gh repo fork breaks.
  • Move from latestLaunch to latestApprovedLaunch when fetching supergraph SDL

@netlify
Copy link

netlify bot commented Apr 9, 2022

👷 Deploy request for apollo-cli-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit c6c780f

@pcarrier pcarrier changed the title hop Use Apollo platform API Apr 9, 2022
@pcarrier pcarrier marked this pull request as ready for review April 9, 2022 03:32
Copy link
Contributor

@EverlastingBugstopper EverlastingBugstopper left a comment

Choose a reason for hiding this comment

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

this looks great!

@EverlastingBugstopper EverlastingBugstopper added this to the 0.6.0 milestone May 2, 2022
@EverlastingBugstopper EverlastingBugstopper merged commit 28c1d35 into apollographql:main May 2, 2022
@pcarrier pcarrier deleted the pcarrier/platform-api branch May 2, 2022 17:43
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.

3 participants