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

docs/node: Fix link to Oasis Cli #613

Merged
merged 1 commit into from
Nov 15, 2023
Merged

docs/node: Fix link to Oasis Cli #613

merged 1 commit into from
Nov 15, 2023

Conversation

amela
Copy link
Member

@amela amela commented Nov 15, 2023

No description provided.

Copy link

netlify bot commented Nov 15, 2023

Deploy Preview for trusting-archimedes-14c863 ready!

Name Link
🔨 Latest commit a3a77a2
🔍 Latest deploy log https://app.netlify.com/sites/trusting-archimedes-14c863/deploys/65548a5a09b7920008826e8e
😎 Deploy Preview https://deploy-preview-613--trusting-archimedes-14c863.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -209,7 +209,7 @@ Ubuntu versions (or other distributions using older system libraries).

11. Start your node and re-enable auto-starting via your process manager.

[Oasis CLI]: ../../general/manage-tokens/cli/
[Oasis CLI]: ../../../general/manage-tokens/cli/
Copy link
Member

Choose a reason for hiding this comment

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

That seems wrong 🤔

This should work:

Suggested change
[Oasis CLI]: ../../../general/manage-tokens/cli/
[Oasis CLI]: ../../general/manage-tokens/cli/README.md

Copy link
Member Author

Choose a reason for hiding this comment

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

But ../../../general/manage-tokens/cli/ works on the preview: https://deploy-preview-613--trusting-archimedes-14c863.netlify.app/node/mainnet/eden-upgrade

Copy link
Member

Choose a reason for hiding this comment

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

Referencing the markdown files instead of URLs is a better idea because it's easier to refactor, links will also work in Github and various IDEs.

Copy link
Member Author

Choose a reason for hiding this comment

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

Also ../../general/manage-tokens/cli/README.md will go to https://docs.oasis.io/node/general/manage-tokens/cli/README.md that does not exists.

@amela amela force-pushed the amela/fix-link-to-cli branch from 618aec6 to a3a77a2 Compare November 15, 2023 09:07
@amela amela merged commit 1a18a9e into main Nov 15, 2023
6 checks passed
@amela amela deleted the amela/fix-link-to-cli branch November 15, 2023 09:14
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