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

MASV docs #388

Merged
merged 2 commits into from
May 29, 2024
Merged

MASV docs #388

merged 2 commits into from
May 29, 2024

Conversation

wthorp
Copy link
Contributor

@wthorp wthorp commented May 29, 2024

No description provided.

Copy link

github-actions bot commented May 29, 2024

Visit the preview URL for this PR (updated for commit b327743):

https://storj-docs--pr388-masv-46yvg9ee.web.app

(expires Wed, 05 Jun 2024 20:33:16 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 3c1f0a7f83da2cd749f45c4149016bdb4af1841c

![](https://link.storjshare.io/raw/jua7rls6hkx5556qfcmhrqed2tfa/docs/images/MASV/Screenshot%202024-05-29%20at%203.59.01%E2%80%AFPM.png)

1. On the **Integrations** tab, enable **Storj** by clicking its corresponding button in the **Status** column.
![](https://link.storjshare.io/raw/jua7rls6hkx5556qfcmhrqed2tfa/docs/images/MASV/Screenshot%202024-05-29%20at%204.01.41%E2%80%AFPM.png)
Copy link
Contributor

@amozoss amozoss May 29, 2024

Choose a reason for hiding this comment

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

Nitpick: If you indent these images 3 spaces under each list item, it'll render the html better for SEO. Aka it'll make the image part of the <li> element instead of separate.

Copy link
Contributor

Choose a reason for hiding this comment

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

Actually, it seems like it does the right thing with how you've done it (no new line in-between and it's directly under the list item). So it's great as is. I learned something :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

welp, I fixed it anyway.

@wthorp wthorp merged commit 533f934 into main May 29, 2024
2 checks passed
@wthorp wthorp deleted the masv branch May 29, 2024 20:38
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.

2 participants