Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RAK14001 LED - Turn on to 50% at boot #2571

Merged
merged 3 commits into from
Jun 27, 2023

Conversation

charminULTRA
Copy link
Sponsor Contributor

@charminULTRA charminULTRA commented Jun 25, 2023

Enables the RAK14001 to turn on to a static 50% white upon boot, and creates a central location (RAKled.h) for defining "rgb".

I could use help in one aspect of this - I don't know C++ fundamentals, so I suspect the organization of code between this and the ExternalNotificationModule could be better organized. I'm not 100% sure if I used best practices, but I tried moving the declaration of "rgb" into the RAKled.h file to give it a more centralized location rather than the Module's file.

In the future, I would like to add the ability to control the LED levels/colors/cycles via the CLI and apps.

@github-actions
Copy link
Contributor

🤖 Pull request artifacts

file commit
pr2571-firmware-2.1.17.c341418.zip c341418

thebentern added a commit to meshtastic/artifacts that referenced this pull request Jun 25, 2023
@charminULTRA
Copy link
Sponsor Contributor Author

charminULTRA commented Jun 25, 2023

I'll also note that I live tested on a RAK 4631

@charminULTRA charminULTRA changed the title Rak14001 LED - Turn on to 50% at boot RAK14001 LED - Turn on to 50% at boot Jun 25, 2023
@thebentern thebentern merged commit 44a906d into meshtastic:master Jun 27, 2023
13 of 24 checks passed
@charminULTRA charminULTRA deleted the RAK14001-LED-Fixed branch June 27, 2023 01:06
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.

None yet

2 participants