Skip to content

Conversation

@bdraco
Copy link
Member

@bdraco bdraco commented Dec 5, 2025

Description:

Update and add buy links for Bluetooth Proxy devices in the web installer:

  • GL.iNet GL-S10: Update store link to working URL and add AliExpress pre-flashed option
  • LilyGO T-ETH-POE: Add official LilyGO Store link

Changes:

  1. Fixed GL.iNet Shop link (old URL was broken)
  2. Added AliExpress link for ESPHome pre-flashed GL-S10 units
  3. Added LilyGO Store as primary buy option for T-ETH-POE board

Related issue (if applicable): N/A

Pull request in esphome with YAML changes (if applicable): N/A

Checklist:

  • I am merging into next because this is new documentation that has a matching pull-request in esphome as linked above.
    or

  • I am merging into current because this is a fix, change and/or adjustment in the current documentation and is not for a new component or feature.

  • Link added in /components/index.rst when creating new documents for new components or cookbook.

Copilot AI review requested due to automatic review settings December 5, 2025 05:34
@esphome esphome bot added the current label Dec 5, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates purchasing links for two Bluetooth Proxy devices in the web installer documentation. The changes fix a broken URL for the GL.iNet GL-S10 and add additional purchasing options for both devices.

Key Changes:

  • Fixed GL.iNet Shop URL for the GL-S10 device
  • Added AliExpress pre-flashed option for GL-S10
  • Added LilyGO Store as primary purchase option for T-ETH-POE

@netlify
Copy link

netlify bot commented Dec 5, 2025

Deploy Preview for esphome ready!

Name Link
🔨 Latest commit 8b8610a
🔍 Latest deploy log https://app.netlify.com/projects/esphome/deploys/69326ed1ecfc0a000865fd78
😎 Deploy Preview https://deploy-preview-5727--esphome.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 5, 2025

Walkthrough

This pull request updates the projects purchase links template by adding alternative AliExpress purchase options for three hardware products: GL.iNet GL-S10, ESP32-GEN, and LilyGO T-ETH-POE ESP32-WROOM, expanding available vendor choices without modifying page logic or structure.

Changes

Cohort / File(s) Summary
Purchase link additions
themes/esphome-theme/layouts/_default/projects.html
Added AliExpress purchase links for three device sections: GL.iNet GL-S10 (AliExpress - ESPHome pre-flashed), Buy ESP32-GEN (AliExpress - ESPHome pre-flashed), and LilyGO T-ETH-POE ESP32-WROOM (AliExpress).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Straightforward content additions with no logic or control flow modifications
  • Changes are localized to HTML list items only
  • No dependencies or functional implications

Suggested labels

current

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main changes: updating and adding buy links for Bluetooth Proxy devices (GL-S10 and LilyGO T-ETH-POE), which directly matches the changeset modifications.
Description check ✅ Passed The description is clearly related to the changeset, detailing specific updates to buy links for GL-S10 and LilyGO T-ETH-POE devices with concrete examples of changes made.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bluetooth_proxy_diy_links

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6132960 and 8b8610a.

📒 Files selected for processing (1)
  • themes/esphome-theme/layouts/_default/projects.html (2 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
**

⚙️ CodeRabbit configuration file

  • Do not generate or add any sequence diagrams

Files:

  • themes/esphome-theme/layouts/_default/projects.html
🧠 Learnings (1)
📚 Learning: 2025-05-12T00:02:50.869Z
Learnt from: jesserockz
Repo: esphome/esphome-docs PR: 4901
File: changelog/2025.4.0.rst:124-126
Timestamp: 2025-05-12T00:02:50.869Z
Learning: In the ESPHome changelog structure, the "All changes" section should have the `:open:` attribute to be expanded by default, but the "Dependency Changes" section should NOT have this attribute as it should remain collapsed by default.

Applied to files:

  • themes/esphome-theme/layouts/_default/projects.html
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Build
🔇 Additional comments (3)
themes/esphome-theme/layouts/_default/projects.html (3)

440-440: Verify GL.iNet Shop URL update and new AliExpress link.

The GL.iNet GL-S10 purchase section has been updated: the GL.iNet Shop URL is now modified (line 440) and a new AliExpress option with "ESPHome pre-flashed" availability has been added (lines 444-449). The changes follow the existing link pattern and provide users with an additional vendor choice.

Please verify that:

  1. The GL.iNet Shop URL (https://store.gl-inet.com/products/gl-s10-ble-iot-gateway?variant=39514685112414) is correct and active.
  2. The AliExpress URL (https://www.aliexpress.us/item/3256803802198078.html) correctly points to an ESPHome pre-flashed GL-S10 variant.

Also applies to: 444-449


564-569: Verify LilyGO Store link.

A new LilyGO Store purchase option has been added as the primary vendor for the LilyGO T-ETH-POE board (lines 564-569). This positions the official store ahead of the existing AliExpress option.

Please verify that the LilyGO Store URL (https://lilygo.cc/en-us/products/t-internet-poe?variant=52074812965045) is correct, active, and links to the correct T-ETH-POE variant.


1-977: Discrepancy in AI summary regarding ESP32-GEN changes.

The AI summary mentions that an AliExpress option was added for "ESP32-GEN," but the provided code shows no changes (marked with ~) in the Generic ESP32 section (lines 406–421).

Please clarify whether:

  1. ESP32-GEN was intended to receive purchase link updates in this PR, or
  2. The AI summary is inaccurate and only GL.iNet GL-S10 and LilyGO T-ETH-POE have been modified.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@swoboda1337 swoboda1337 merged commit 1a80329 into current Dec 5, 2025
12 checks passed
@swoboda1337 swoboda1337 deleted the bluetooth_proxy_diy_links branch December 5, 2025 13:40
Undearius pushed a commit to Undearius/esphome-docs that referenced this pull request Dec 6, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Dec 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants