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(snapshot.md): fixed broken urls again using absolute path #195

Merged
merged 1 commit into from
Jan 21, 2024

Conversation

vikgaur
Copy link
Contributor

@vikgaur vikgaur commented Jan 19, 2024

No description provided.

@@ -29,7 +29,7 @@ Currently, Mayastor supports the following operations related to volume snapshot
## Prerequisites


1. Deploy and configure Mayastor by following the steps given [here](quickstart/deploy-mayastor.md) and create disk pools.
1. Deploy and configure Mayastor by following the steps given [here](https://mayastor.gitbook.io/introduction/quickstart/deploy-mayastor.md) and create disk pools.
Copy link
Contributor

Choose a reason for hiding this comment

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

Previous PR had used relative path, example: ../
If that works it'd be better IMHO

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've tried using the relative path but it's not working. So, I've used the absolute path to fix the issue.

Copy link
Member

Choose a reason for hiding this comment

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

The relative path change is already live, but not working. Not sure of the reason. Also noticed that elsewhere in the docs, we seem to have used absolute paths (except SUMMARY which is at top-level).

@avishnu
Copy link
Member

avishnu commented Jan 19, 2024

There is a conflict in your branch that needs to be resolved. Most likely, you haven't rebased your local branch with the remote.

@vikgaur
Copy link
Contributor Author

vikgaur commented Jan 19, 2024

@avishnu @tiagolobocastro I've fixed the conflict. Please review.

Copy link
Member

@avishnu avishnu left a comment

Choose a reason for hiding this comment

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

lgtm

@avishnu avishnu merged commit 8e944f8 into develop Jan 21, 2024
3 checks passed
@avishnu avishnu deleted the broken-url branch January 21, 2024 14:49
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