Skip to content
This repository has been archived by the owner on Jul 16, 2024. It is now read-only.

Update balena etcher references to point to 1.18.11 closes PhotonVision/photonvision-docs#357 #358

Merged
merged 8 commits into from
May 30, 2024

Conversation

Sam948-byte
Copy link
Contributor

Update links to balena etcher to point to 1.18.11 and change the warning against older versions to only warn against versions older than 1.18.11, as it is not the newest version.

mcm001
mcm001 previously approved these changes May 30, 2024
Copy link
Contributor

@mcm001 mcm001 left a comment

Choose a reason for hiding this comment

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

Lgtm 🛥️

@Sam948-byte
Copy link
Contributor Author

PR is finalized

@Sam948-byte Sam948-byte changed the title Update balena etcher references to point to 1.18.11 Update balena etcher references to point to 1.18.11 closes #357 May 30, 2024
@Sam948-byte Sam948-byte changed the title Update balena etcher references to point to 1.18.11 closes #357 Update balena etcher references to point to 1.18.11 closes PhotonVision/photonvision-docs#357 May 30, 2024
@mcm001
Copy link
Contributor

mcm001 commented May 30, 2024

@Sam948-byte
Copy link
Contributor Author

I replaced this link with this one

I couldn't find the second broken link, so if you could point that one out, that would be great.

@Sam948-byte
Copy link
Contributor Author

Sam948-byte commented May 30, 2024

I found the second one, it looks like it's this link. It is not passing the check, but the link is functional, both from the build of the documentation and by copying and pasting into a browser. I'm not quite sure why it's failing the check.

As it's giving a 403, I'm assuming that something specific to the check means it can't access it, perhaps the raspberry pi site has some sort of captcha. I don't believe this affects anything, however, and I think we should be good to go.

@mcm001 mcm001 merged commit 5fa174f into PhotonVision:master May 30, 2024
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants