Skip to content

[docs] fix broken links #731

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

Closed
wants to merge 2 commits into from

Conversation

zjbob
Copy link

@zjbob zjbob commented Jun 26, 2023

Description

I was reading through the 0x docs and noticed a broken link, so this PR fixes the broken link to the proxy page from the feature page.

I have not included any tests or changelog updates for this PR. Please let me know if/where I should add any of these.

Testing instructions

To my knowledge, not much testing can be done until the updated version of these docs are deployed. However since governor.rst and proxy.rst are in the same directory, you can see that the relative path for proxy is now identical to the relative path for governor in this screenshot:

0xdocfix

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • Add new entries to the relevant CHANGELOG.jsons.

@zjbob zjbob marked this pull request as ready for review June 26, 2023 20:05
@zjbob zjbob changed the title [docs] fix broken link in features.rst [docs] fix broken links Jun 27, 2023
@zjbob
Copy link
Author

zjbob commented Jun 29, 2023

@wonge97 Is there anything I should do about these failed CI tests? Are they really failing just from updating the docs?

@wonge97
Copy link
Collaborator

wonge97 commented Jul 5, 2023

hey @Abyiss-Zach , sorry for late reply frokm me - I don't think there's anything to do about the failed CI tests, wouldn't be related to the doc updates here.

@zjbob
Copy link
Author

zjbob commented Jul 6, 2023

@wonge97 No worries at all. Now since these CI tests have failed, I am unable to merge this PR in. It also appears that I am unable to rerun any of these workflows. Is there a procedure somewhere or something that I should perform to get this PR merged?

@github-actions
Copy link
Contributor

This pull request 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants