Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Broken ReDoc Link #900

Closed
1 task done
ramadanomar opened this issue Aug 27, 2022 · 3 comments
Closed
1 task done

Broken ReDoc Link #900

ramadanomar opened this issue Aug 27, 2022 · 3 comments
Assignees
Labels
📄 aspect: text Concerns the textual material in the repository 🛠 goal: fix Bug fix 🟩 priority: low Low priority and doesn't need to be rushed

Comments

@ramadanomar
Copy link
Contributor

Description

qucikstart.md has a broken link to Api Redoc page.

Screenshots image

Resolution

  • 🙋 I would be interested in resolving this bug.

If the ReDoc documentation is this one, i'll create a pr to update it. Please lmk if this is an old/outdated link as i couldn't find anything newer on the internet/slack

@ramadanomar ramadanomar added 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work 🛠 goal: fix Bug fix labels Aug 27, 2022
@ramadanomar
Copy link
Contributor Author

Here the port changed from 8000 to 50280 which no longer points to the docs. I cant find where the doc content is located to edit it. The quickstart.md has the new port (50280). I'll try to look into it and update pr #901
image

@obulat
Copy link
Contributor

obulat commented Aug 29, 2022

@ramadanomar, thank you for reporting this!

It's an interesting issue. The broken link is actually a screenshot image that is available in the repository. You can see it load on the Sphinx documentation page here: https://wordpress.github.io/openverse-api/guides/quickstart.html. However, the Sphinx doc page still has the old port information. @dhruvkb, do we need to re-build the docs to update the ports? I wonder if we will see this broken image on the docs page after we rebuild it?

@obulat obulat added 🟩 priority: low Low priority and doesn't need to be rushed 📄 aspect: text Concerns the textual material in the repository and removed 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work labels Aug 29, 2022
@ramadanomar
Copy link
Contributor Author

The documentation page appears to have the correct port now. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
📄 aspect: text Concerns the textual material in the repository 🛠 goal: fix Bug fix 🟩 priority: low Low priority and doesn't need to be rushed
Projects
None yet
Development

No branches or pull requests

2 participants