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

Link to First Part of "Golang + GraphQL + Relay" Blog Posts #497

Merged
merged 2 commits into from
Feb 5, 2020

Conversation

EIrwin
Copy link
Contributor

@EIrwin EIrwin commented Jul 28, 2019

I noticed that this just links to the blog, and a couple of times, it made me think the link was broken and I ended up leaving not knowing to scroll down and still find the relevant blog.

I think it would be helpful to change this to link to the first post of the relevant blog series instead.

@coveralls
Copy link

coveralls commented Jul 28, 2019

Coverage Status

Coverage remained the same at 92.376% when pulling 11836f4 on EIrwin:master into 77807ed on graphql-go:master.

Copy link
Member

@chris-ramon chris-ramon left a comment

Choose a reason for hiding this comment

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

LGTM 👍 –– Nice catch @EIrwin, thanks a lot!

@chris-ramon chris-ramon merged commit 69bb0b7 into graphql-go:master Feb 5, 2020
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