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

Fix broken links in APIM docs #8817

Closed
RusJaI opened this issue Nov 12, 2024 · 3 comments · Fixed by #8818
Closed

Fix broken links in APIM docs #8817

RusJaI opened this issue Nov 12, 2024 · 3 comments · Fixed by #8818
Assignees

Comments

@RusJaI
Copy link
Contributor

RusJaI commented Nov 12, 2024

Run a broken links scanning tool in the new doc space and do the related fixes.

Check the first link here [1]. It has linked to the following URL [2] which resulted in the page not found. Same link works in 4.2.0, but not in 4.3.0 or 4.4.0.
[1] https://apim.docs.wso2.com/en/latest/design/create-api/create-an-api-using-a-service/#step-1-create-and-publish-your-service-to-the-service-catalog
[2] https://apim.docs.wso2.com/en/4.4.0/integrate/develop/working-with-service-catalog

Email : Re: Update on [Document improvement ]Progress of Broken links fixing

@RusJaI
Copy link
Contributor Author

RusJaI commented Nov 12, 2024

Update [2024-11-12]

scanned the doc space using [1] and fixed the broken links with commit 2c56a48

However there are some docs from MI which contain broken links. As per an internal discussion, it was recommended to remove those docs from docspace. Since there are number of usages of those docs, that need to be removed carefully.

[1] https://www.brokenlinkcheck.com/

@RusJaI
Copy link
Contributor Author

RusJaI commented Nov 13, 2024

Update [2024-11-13]

MI related docs have been removed by an parallel effort, tracked by [1].
Currently executing another 2 scanners [2] and [3] and will fix the suggested fixes accordingly.

[1] #8727
[2] https://www.drlinkcheck.com/
[3] https://www.deadlinkchecker.com/

added commit 3a50ccf to fix the identified cases by the above scanners.

@RusJaI
Copy link
Contributor Author

RusJaI commented Nov 14, 2024

Update [2024-11-14]

APIM 4.4.0 Fix : #8825

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

Successfully merging a pull request may close this issue.

1 participant