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

Introduce the ability to add multiple wireless network configurations #12

Merged
merged 1 commit into from
Nov 29, 2018

Conversation

tmecklem
Copy link
Contributor

Related to nerves-project-attic/nerves_network#72.

Add add_network/2, remove_all_networks/1, and reassociate/1
functions to give more fine grained control over the configuration and
activation process while shielding callers from implementation details,
and reworks set_network/2 to use the new methods.

Add documentation for :priority which is useful when configuring
Multiple networks.

Add `add_network/2`, `remove_all_networks/1`, and `reassociate/1`
functions to give more fine grained control over the configuration and
activation process while shielding callers from implementation details,
and reworks `set_network/2` to use the new methods.

Add documentation for `:priority` which is useful when configuring
Multiple networks.
Copy link
Contributor

@fhunleth fhunleth left a comment

Choose a reason for hiding this comment

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

I didn't test this out, but I like what you did. This is a nice improvement!

@newellista
Copy link

What are the chances of getting this merged into master soon?

@fhunleth
Copy link
Contributor

@ConnorRigby

@fhunleth fhunleth requested a review from ConnorRigby November 20, 2018 12:58
@ConnorRigby ConnorRigby merged commit 3381a0c into master Nov 29, 2018
@ConnorRigby ConnorRigby deleted the add-network branch November 29, 2018 18:53
@ConnorRigby
Copy link
Contributor

Sorry for taking so long to get this merged. Looks great to me.

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