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

fastify-apollo has been merged into apollo-server #1126

Closed
wants to merge 1 commit into from
Closed

fastify-apollo has been merged into apollo-server #1126

wants to merge 1 commit into from

Conversation

dtex
Copy link

@dtex dtex commented Aug 30, 2018

As of this PR, fastify-apollo is no longer a standalone module and the repo has been archived.

@nfishe Can you add a note on your repo also? It would be helpful for anyone who stumbles across it.

Checklist

  • run npm run test and npm run benchmark
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message and code follows Code of conduct

As of [this PR](apollographql/apollo-server#1013), fastify-apollo is no longer a standalone module and the repo has been archived.

@nfishe Can you add a note on your repo also? It would be helpful for anyone who stumbles across it.
@mcollina
Copy link
Member

Awesome! I'll merge this once that is released, is that ok?

https://www.npmjs.com/package/apollo-server-fastify is still 10 months old.

Copy link
Member

@delvedor delvedor left a comment

Choose a reason for hiding this comment

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

I think we can leave the line in our documentation and update the url to the new repo :)

@delvedor delvedor added the documentation Improvements or additions to documentation label Aug 30, 2018
@dtex
Copy link
Author

dtex commented Aug 30, 2018

Is that ok?

Of course. I was just hoping to save someone the head-scratching that I just experienced. This PR doesn't really help given that

https://www.npmjs.com/package/apollo-server-fastify is still 10 months old.

Oh... well... darn. I guess we are in some kind of valley between fastify-apollo and apollo-server-fastify.

leave the line in our documentation and update the url

@delvedor I think that's fine as long as there is an explanatory note. It would be confusing if the link text says you're going one place and then the link goes to another. However, given Mateo's note, things are a little more complicated than I understood them to be. We may as well close this rather than leave it open indefinitely.

@delvedor
Copy link
Member

@dtex what I mean is, instead of removing the line, just update it with the new plugin :)

@dtex
Copy link
Author

dtex commented Aug 30, 2018

@delvedor Oh, gotchya. Yes, once it goes live that is the right thing to do.

@mannyluvstacos
Copy link

Hi, @dtex and all!

It would be helpful for anyone who stumbles across it.

I was one of those people that stumbled upon the fastify-apollo module and saw that it was archived and also saw: Integrate apollo-fastify plugin #626 #1013.

For my understanding, there is a PR in flight to create a apollo-server-fastify and fastify-apollo is to no longer be used?

I look forward to hearing back!

-Manny

@stale
Copy link

stale bot commented Sep 19, 2018

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.

@stale stale bot added the stale Issue or pr with more than 15 days of inactivity. label Sep 19, 2018
@stale stale bot closed this Sep 26, 2018
@cemremengu cemremengu reopened this Sep 26, 2018
@stale stale bot removed the stale Issue or pr with more than 15 days of inactivity. label Sep 26, 2018
@dtex
Copy link
Author

dtex commented Oct 5, 2018

Just an update. There is a new PR for the support we need in Apollo:

apollographql/apollo-server#1760

@stale
Copy link

stale bot commented Oct 20, 2018

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.

@stale stale bot added the stale Issue or pr with more than 15 days of inactivity. label Oct 20, 2018
@cemremengu cemremengu added discussion Issues or PRs with this label will never stale and removed stale Issue or pr with more than 15 days of inactivity. labels Oct 20, 2018
@delvedor
Copy link
Member

delvedor commented Mar 9, 2019

What's the status on this?

@darkgl0w
Copy link
Member

darkgl0w commented Mar 10, 2019

Since the release of apollo-server@2.4.2, fastify support has officially landed (related PR: apollographql/apollo-server#2280)

I think this PR just needs an update to reference the official package (by pointing to this link : https://github.com/apollographql/apollo-server/tree/master/packages/apollo-server-fastify and referencing the package as apollo-server-fastify) ^^

Edit: there is a related PR running here : #1156

@delvedor
Copy link
Member

@darkgl0w would you like to open a new pr with the correct link?

@dtex
Copy link
Author

dtex commented Mar 10, 2019

Thank you @darkgl0w Closing this in deference to #1524

@dtex dtex closed this Mar 10, 2019
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
discussion Issues or PRs with this label will never stale documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants