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

fixed broken links to graphene filter documentation (master->main) #1309

Merged
merged 1 commit into from
Feb 12, 2022
Merged

Conversation

ppk42
Copy link
Contributor

@ppk42 ppk42 commented Feb 12, 2022

This is a follow up of PR #1296. I unfortunatly delete my fork bfore the PR was reviewed and approved and, I had not used the most wanted branch name in the links. And I did not manage it to reassigne the new fork to the former PR #1296.

@ulgens ulgens self-requested a review February 12, 2022 14:26
@ulgens
Copy link
Collaborator

ulgens commented Feb 12, 2022

@ppk42 Thanks for your effort! 🚀

@ulgens ulgens merged commit bf8fd76 into graphql-python:main Feb 12, 2022
@ulgens
Copy link
Collaborator

ulgens commented Feb 12, 2022

@ppk42 Do we have the same issue in v2 branch?

@ppk42
Copy link
Contributor Author

ppk42 commented Feb 12, 2022

@ulgens You are right. V2 is affected too. I've also overseen a possibly wrong link to the github master tree. The master branch still exists but code in the main branch should refer to the main tree, right? I'll add the necessary PR. Thank you for the review.

ulgens pushed a commit that referenced this pull request Feb 13, 2022
* fix broken links for the v2 branch

v2 brach has broken links to read the docs too

I additionally found a link to the git hub master tree, which should be
changed to main.

* #1295 github link fixed (master->v2)

Co-authored-by: Peter Paul Kiefer <dafisppk@gmail.com>
ulgens pushed a commit that referenced this pull request Feb 13, 2022
* fixed broken links to graphene filter documentation (master->main)

* #1295 There is still a wrong link to github

The referenced example is in main branch
but the link goes to the master branch which still exists.

Co-authored-by: Peter Paul Kiefer <dafisppk@gmail.com>
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.

2 participants