Fix incorrect usermod name: LD2410 -> LD2410_v2#4756
Conversation
Compiling doesn't work because it doesn't find LD2410 in usermods. The correct usermod is LD2410_v2
WalkthroughThe documentation for enabling the LD2410 usermod was updated to correct the custom usermod name in the example configuration, changing it from Changes
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (2)📓 Common learningsusermods/LD2410_v2/readme.md (1)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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
thanks! |
Compiling doesn't work because it doesn't find LD2410 in usermods. The correct usermod is LD2410_v2
|
It was driving me mad and I was thinking I am stupid. Blindly copy-pasted the string from the readme and get an error ' The usermod couldn't be found'. What really was is the name in the readme is incorrect. Please change LD2140_v2 to LD2410_v2 |
|
@KotBayun69 you can provide a patch. |
Compiling the WLED firmware with the
LD2410usermod fails because there is no module with that name in theusermodsdirectory.The correct usermod directory is
LD2410_v2, so the usermod name must match that exactly in the build configuration.This fix updates the example or documentation (depending on what you're editing) to reflect the correct usermod name.
Tested with:
platformio_override.iniusing: