Skip to content

Add pico_w/wifi/access_point_wifi_provisioning/#754

Open
peterharperuk wants to merge 2 commits into
raspberrypi:developfrom
peterharperuk:add_access_point_wifi_provisioning
Open

Add pico_w/wifi/access_point_wifi_provisioning/#754
peterharperuk wants to merge 2 commits into
raspberrypi:developfrom
peterharperuk:add_access_point_wifi_provisioning

Conversation

@peterharperuk

Copy link
Copy Markdown
Contributor

No description provided.

@peterharperuk peterharperuk added this to the 2.2.1 milestone Jun 5, 2026
Louis31423142 and others added 2 commits June 11, 2026 10:39
Fix compile warning.
Use the same password as the other access point example.
Fix captive portal
@peterharperuk peterharperuk force-pushed the add_access_point_wifi_provisioning branch from a181961 to 6716136 Compare June 11, 2026 09:58
@kilograham kilograham requested review from lurch and will-v-pi June 17, 2026 22:39

@will-v-pi will-v-pi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This doesn't seem to handle SSIDs with spaces correctly, because the form submission passes them as + instead, so it tries to connect to "My+Network" instead of "My Network". This might be an issue for other special characters too.

@kilograham kilograham self-requested a review June 18, 2026 19:43

@kilograham kilograham left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

un-approving to allow for will's comment

@peterharperuk

Copy link
Copy Markdown
Contributor Author

This example relies on this change raspberrypi/pico-sdk#3000

@will-v-pi

Copy link
Copy Markdown
Contributor

This example relies on this change raspberrypi/pico-sdk#3000

I'm still seeing the issue with SSIDs with spaces now that change has been merged - do SSIDs with spaces work for you?

@peterharperuk

Copy link
Copy Markdown
Contributor Author

I haven't looked into that problem yet

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.

4 participants