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.

Sign up for GitHub

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

Jump to bottom

Docs: Minor fixes to edit link & clarifications #501

Merged
merged 3 commits into from Mar 20, 2024
Merged

Docs: Minor fixes to edit link & clarifications #501

merged 3 commits into from Mar 20, 2024

Conversation

Shrinks99
Copy link
Member

@Shrinks99 Shrinks99 commented Mar 20, 2024 •
edited

Changes

  • Fixes docs edit link GitHub path, unlike Browsertrix everything lives in a separate /docs directory here!
  • Adds a note directing people to Browsertrix — the cloud platform — and its documentation to the homepage

@Shrinks99 Shrinks99 added the documentation Improvements or additions to documentation label Mar 20, 2024
@Shrinks99 Shrinks99 requested a review from tw4l March 20, 2024 16:43
tw4l
tw4l reviewed Mar 20, 2024
View reviewed changes
docs/docs/index.md Outdated Show resolved Hide resolved
tw4l
tw4l approved these changes Mar 20, 2024
View reviewed changes
Copy link
Contributor

@tw4l tw4l left a comment

Choose a reason for hiding this comment

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

Thanks! One suggestion, I'll leave it up to you whether to take or leave it.

@Shrinks99 @tw4l
Update docs/docs/index.md
3ec9d1b 
Co-authored-by: Tessa Walsh <tessa@bitarchivist.net>
@Shrinks99 Shrinks99 merged commit 79e39ae into main Mar 20, 2024
3 of 4 checks passed
@Shrinks99 Shrinks99 deleted the docs-minor-fixes branch March 20, 2024 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tw4l tw4l tw4l approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Shrinks99 @tw4l