Skip to content

Conversation

MabezDev
Copy link
Member

@MabezDev MabezDev commented Oct 1, 2025

Drafting as we need to do the releases first

@MabezDev MabezDev marked this pull request as draft October 1, 2025 11:30
@MabezDev MabezDev requested a review from bjoernQ October 1, 2025 11:30
} else if (path.startsWith("/esp-hal/esp-wifi")) {
window.location.href = "https://docs.espressif.com/projects/rust/esp-wifi/latest/";
window.location.href = "https://docs.espressif.com/projects/rust/esp-radio/latest/";
} else if (path.startsWith("/esp-hal/esp-hal-embassy") || path.startsWith("/projects/rust/esp-hal-embassy")) {
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think there are any links out there matching path.startsWith("/projects/rust/esp-hal-embassy") 🤔

Copy link
Member Author

Choose a reason for hiding this comment

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

https://docs.espressif.com/projects/rust/esp-hal-embassy/0.9.0/index.html will 404 once we remove the package (well, I guess it won't, but we might remove those files at some point in the future), so I added it here.

Copy link
Contributor

Choose a reason for hiding this comment

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

ah yes - but we only get to this 404 handler via https://esp-rs.github.io/ - I think it's something for esp-rs/esp-hal#3388

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh yeah, of course 🤦‍♂️. I'll drop a comment in the PR for esp-rs/esp-hal#3388 and let's see if we can add it there.

Copy link
Contributor

@bjoernQ bjoernQ left a comment

Choose a reason for hiding this comment

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

looks good if we remove the /projects/rust/esp-hal-embassy check (it doesn't hurt but looks confusing)

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