Skip to content

feat(eps-now): add 3 scenes for GLEDOPTO ESP-NOW WLED Remote Control#5400

Merged
DedeHai merged 4 commits intowled:mainfrom
GLEDOPTO:main
Mar 3, 2026
Merged

feat(eps-now): add 3 scenes for GLEDOPTO ESP-NOW WLED Remote Control#5400
DedeHai merged 4 commits intowled:mainfrom
GLEDOPTO:main

Conversation

@GLEDOPTO
Copy link
Contributor

@GLEDOPTO GLEDOPTO commented Feb 27, 2026

This PR adds three new scenes to the EPS-NOW remote controller to support the GLEDOPTO ESP-NOW WLED Remote Control.

Product link: https://gledopto.com/h-pd-146.html

Summary by CodeRabbit

  • New Features
    • Added support for three new WizMote remote buttons (5, 6, 7) for one-press access to lighting presets: Candle, Random Color, and Fade for quicker mode changes.

This PR adds three new scenes to the EPS-NOW remote controller to support the GLEDOPTO ESP-NOW WLED Remote Control.

Product link: https://www.amazon.com/dp/B0G7YZ5VJQ
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between fa13a7c and 6e06516.

📒 Files selected for processing (1)
  • wled00/remote.cpp
🚧 Files skipped from review as they are similar to previous changes (1)
  • wled00/remote.cpp

Walkthrough

Added three WizMote button macros and switch handling in wled00/remote.cpp for buttons FIVE, SIX, SEVEN. Each case calls presetWithFallback(..., palette=0, presetId=5|6|7), mapping to preset effects FX_MODE_CANDLE, FX_MODE_RANDOM_COLOR, and FX_MODE_FADE respectively.

Changes

Cohort / File(s) Summary
WizMote button additions
wled00/remote.cpp
Added macros WIZMOTE_BUTTON_FIVE, WIZMOTE_BUTTON_SIX, WIZMOTE_BUTTON_SEVEN (20, 21, 22) and switch cases invoking presetWithFallback with palette=0 and presetId=5, 6, 7, mapping to FX_MODE_CANDLE, FX_MODE_RANDOM_COLOR, FX_MODE_FADE.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • softhack007
  • DedeHai
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title mentions 'GLEDOPTO ESP-NOW WLED Remote Control' and '3 scenes', but the changes actually add 3 new button definitions (FIVE, SIX, SEVEN) that map to presets/effects, not 'scenes'. This is a terminology mismatch between the title and the actual implementation. Clarify whether these are 'scenes' or 'button presets/effects', and ensure the title accurately reflects the technical implementation (e.g., 'add 3 new remote buttons for GLEDOPTO ESP-NOW controller').
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

GLEDOPTO added 2 commits March 2, 2026 13:44
This PR adds three new scenes to the EPS-NOW remote controller to support the GLEDOPTO ESP-NOW WLED Remote Control.

Product link: https://gledopto.com/h-pd-146.html
This PR adds three new scenes to the EPS-NOW remote controller to support the GLEDOPTO ESP-NOW WLED Remote Control.

Product link: https://gledopto.com/h-pd-146.html
@GLEDOPTO GLEDOPTO requested a review from DedeHai March 2, 2026 05:51
This PR adds three new scenes to the EPS-NOW remote controller to support the GLEDOPTO ESP-NOW WLED Remote Control.

Product link: https://gledopto.com/h-pd-146.html
@DedeHai DedeHai merged commit c604b99 into wled:main Mar 3, 2026
softhack007 pushed a commit to MoonModules/WLED-MM that referenced this pull request Mar 3, 2026
maps codes 20, 21 and 22 to presets 5, 6 and 7
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