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

Added script and workflows to check broken links #2819

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

logu1411
Copy link
Contributor

No description provided.

Copy link
Contributor

Preview available here

Copy link
Contributor

The following broken links were found:

Copy link
Contributor

Choose a reason for hiding this comment

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

Would it be good to place it in the dedicated folder, instead of generic scripts? Like it's done for add-release-dates tool script.

Comment on lines +27 to +32
"link-sag-portal": "https://empower.softwareag.com/",
"link-sag-tech-forum": "https://tech.forums.softwareag.com/",
"link-apamadoc-api": "https://documentation.softwareag.com/pam/10.15.5/en/webhelp/related/ApamaDoc/",
"link-sag-dev-community": "https://tech.forums.softwareag.com/tag/Cumulocity-IoT",
"link-apama-webhelp": "https://documentation.softwareag.com/pam/10.15.5/en/webhelp/pam-webhelp/",
"link-sag-privacy-statement": "https://www.softwareag.com/en_corporate/privacy.html",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we consider .softwareag.com links as "broken", even if technically they are still working?

Copy link
Collaborator

Choose a reason for hiding this comment

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

These are outdated and do no longer show up in the develop branch.

@BeateRixen
Copy link
Collaborator

@logu1411 The links to download.cumulocity.com are not broken but require credentials.

@@ -0,0 +1,19 @@
URL,File Path,Status Code
https://github.com/Cumulocity-IoT/cumulocity-examples/snmp,..\content\change-logs\device-management\SNMP-removed-from-docs.md,404
https://download.cumulocity.com/,..\content\about_documentation\scope.md,Error
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess https://download.cumulocity.com links are fine, they just require login, which might caused error here

https://github.com/Cumulocity-IoT/cumulocity-examples/snmp,..\content\change-logs\device-management\SNMP-removed-from-docs.md,404
https://download.cumulocity.com/,..\content\about_documentation\scope.md,Error
https://download.cumulocity.com/,..\content\additional-resources\more-documentation.md,Error
https://portal.azure.com/,..\content\datahub\integrating-datahub-with-other-products-bundle\integration-powerbi.md,403
Copy link
Contributor

Choose a reason for hiding this comment

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

Requires login, but might be actually correct.

https://download.cumulocity.com/,..\content\about_documentation\scope.md,Error
https://download.cumulocity.com/,..\content\additional-resources\more-documentation.md,Error
https://portal.azure.com/,..\content\datahub\integrating-datahub-with-other-products-bundle\integration-powerbi.md,403
https://github.com/eclipse/paho.mqtt.python/issues/354,..\content\device-integration\mqtt-examples-bundle\hello-mqtt-python.md,404
Copy link
Contributor

Choose a reason for hiding this comment

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

It opens fine for browser user.

https://download.cumulocity.com/,..\content\edge\edge-databroker-bundle\databroker-edge.md,Error
https://download.cumulocity.com/,..\content\edge\edge-update-bundle\updating-edge-gui.md,429
https://download.cumulocity.com/,..\content\edge\edge-installation-bundle\prerequisites.md,429
https://www.mathworks.com/help/predmaint/ug/remaining-useful-life-estimation-using-convolutional-neural-network.html,..\content\machine-learning\AI-ML-integration-bundle\step1-create-ai-ml-model.md,403
Copy link
Contributor

Choose a reason for hiding this comment

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

works for browser user

Comment on lines +13 to +14
http://www.oracle.com/technetwork/java/javase/downloads/index.html,..\content\microservice-sdk\java-bundle\introduction.md,403
https://www.oracle.com/technetwork/java/javase/downloads/index.html,..\content\microservice-sdk\java-bundle\ip-tracker-microservice.md,403
Copy link
Contributor

Choose a reason for hiding this comment

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

They redirect but in the end reach download page.

https://en.wikipedia.org/wiki/Sampling_(signal_processing,..\content\streaming-analytics\analytics-builder-bundle\details-of-values-and-blocks.md,404
https://en.wikipedia.org/wiki/Quantization_(signal_processing,..\content\streaming-analytics\analytics-builder-bundle\details-of-values-and-blocks.md,404
https://cumulocity.com/docs/<https://en.wikipedia.org/wiki/Glob_(programming,..\content\web\upgrade-bundle\c8y-cli.md,404
https://code.visualstudio.com/,..\content\web\upgrade-bundle\updating-web-sdk-version.md,500
Copy link
Contributor

Choose a reason for hiding this comment

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

It's not broken for browser user.

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