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

RasPi3-WiFi #57

Merged
merged 3 commits into from
Mar 20, 2017
Merged

RasPi3-WiFi #57

merged 3 commits into from
Mar 20, 2017

Conversation

PureGrain
Copy link
Contributor

This will help support built in wifi/wlan on RasPi 3. I've tested it on 12+ devices and it works great.

Added support for Raspberry Pi 3 on board wifi/wlan adapter.
Added support for RasPi 3 on board Wifi
@jrmhaig
Copy link
Collaborator

jrmhaig commented Feb 7, 2017

Thanks, this looks useful. Can you please clarify:

  • Are these settings likely to break things for other wireless devices? If not then perhaps they can be turned on by default.
  • Would it be helpful to have these options configurable in the UI? Or is it more sensible to try to detect the device and then try to set defaults accordingly? I haven't got as far as thinking how this would be done.

I will merge this soon but I will wait for a short while so that this remains more visible.

@PureGrain
Copy link
Contributor Author

I will test it with a couple of wireless dongles later today and report back. I don't think it would need to be configurable unless it doesn't work. As far as setting accordingly from detection, it wouldn't be hard considering that part of the code is already present in the System page to present which version of the Pi is being used.

getveloce added a commit to getveloce/raspap-webgui that referenced this pull request Feb 25, 2017
@PureGrain
Copy link
Contributor Author

I have tested this with a Belkin and Edimax Dongle and it works fine. I might add that it also works with the Raspberry Pi Zero W (wireless). I just finished the tests and all works well.

@jrmhaig jrmhaig merged commit 451b03a into RaspAP:master Mar 20, 2017
@jrmhaig
Copy link
Collaborator

jrmhaig commented Mar 20, 2017

Thanks.

@billz
Copy link
Member

billz commented May 19, 2017

Closing related issue #39. Thanks for the addition @PureGrain

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

3 participants