Skip to content

Allow to use current wifi settings if location permission is not yet given#336

Closed
ducker wants to merge 1 commit into
eerimoq:mainfrom
ducker:feature/wifi-credentials
Closed

Allow to use current wifi settings if location permission is not yet given#336
ducker wants to merge 1 commit into
eerimoq:mainfrom
ducker:feature/wifi-credentials

Conversation

@ducker

@ducker ducker commented May 16, 2026

Copy link
Copy Markdown
Contributor

Current implementation autofills the field only when location permission is already given. I changed that and added a button which ask for permission if not yet given and allow to fill textfield with WiFi SSID. It also preselects stream url.

@eerimoq

eerimoq commented May 16, 2026

Copy link
Copy Markdown
Owner

One feature per PR. Looks like a lot of code for a smallish improvement? Any chance you can refactor it to make it easier to understand?

@ducker

ducker commented May 16, 2026

Copy link
Copy Markdown
Contributor Author

In my opinion it's one feature. Added button that allow to ask user for location permission if not granted prior and fills the configuration details. Do you have some idea how to split it into smaller pieces?

Most of the added lines comes from localisation file as there are some toasts information for the user to better understand the process.

@eerimoq

eerimoq commented May 17, 2026

Copy link
Copy Markdown
Owner

automatically filling in the password seems like a good first PR, but instead of doing it when an SSID has been discovered, do it in the SSID submit callback. I'm not yet sure all other changes should be merged. have to think about it. last week we removed some of the auto selecting as it was confusing to streamers, not knowing why their settings randomly changed. we may add more magic again, but let's not rush it

@eerimoq

eerimoq commented Jun 1, 2026

Copy link
Copy Markdown
Owner

we might add all of it soonish. had some feedback at twitchcon =)

@eerimoq eerimoq closed this Jul 5, 2026
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