Skip to content
This repository was 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

samfreund
Copy link
Member

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 🛥️

@samfreund
Copy link
Member Author

PR is finalized

@samfreund samfreund 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
@samfreund samfreund 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

@samfreund
Copy link
Member 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.

@samfreund
Copy link
Member Author

samfreund 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