Skip to content

feat(wifi): add settings Wi-Fi provisioning - #4

Open
xuruiray wants to merge 1 commit into
m5stack:mainfrom
xuruiray:xurui/stopwatch-wifi-settings
Open

xuruiray wants to merge 1 commit into
m5stack:mainfrom
xuruiray:xurui/stopwatch-wifi-settings

Conversation

@xuruiray

Copy link
Copy Markdown

Summary

  • Add a HAL Wi-Fi controller that coordinates station mode, setup AP, and Badge AP ownership.
  • Add Settings > Device > Wi-Fi with enable/disable control and status text for connecting, connected, setup AP, and Badge AP states.
  • Add a centered Wi-Fi indicator to the status bar next to the battery icon and battery level.

Test plan

  • git diff --check upstream/main..HEAD
  • idf.py reconfigure
  • idf.py build

Notes

  • The final PR branch is rebased onto m5stack/main and contains one Wi-Fi upgrade commit.
  • Earlier device flashing/monitoring was performed while developing the same feature before rebasing to upstream; this final upstream-based commit was build-verified after the rebase.

Summary:
- add a HAL Wi-Fi controller for STA, setup AP, and Badge AP coordination
- add Settings > Device > Wi-Fi with connection/config AP status
- add a centered Wi-Fi indicator to the status bar

Rationale:
- keep Wi-Fi ownership centralized so Settings and Badge AP do not race over esp_wifi state
- keep LVGL updates on the UI polling path instead of Wi-Fi event callbacks

Tests:
- git diff --check upstream/main..HEAD
- idf.py reconfigure
- idf.py build

Co-authored-by: Codex <codex@openai.com>
@xuruiray

Copy link
Copy Markdown
Author

Hi @Forairaaaaa, could you please take a look at this Wi-Fi settings/provisioning PR when you have a chance? Thanks!

@Forairaaaaa

Copy link
Copy Markdown
Contributor

Hi @Forairaaaaa, could you please take a look at this Wi-Fi settings/provisioning PR when you have a chance? Thanks!

Thanks! I'll take a look.

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