refactor(#495): method to discover hidden networks (opt-in)#496
refactor(#495): method to discover hidden networks (opt-in)#496justinjest wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
I appreciate your contributions- could you just read the contribution guidelines and make sure you follow things like the commit message hygiene? You should also reference the issue you are closing in your PR description (i.e. "Closes #495" so that it properly links the issue as being closed by this PR once merged).
| pub wifi_groups: Vec<WifiNetworkGroup>, | ||
| /// All Wi-Fi groups keyed by interface and SSID. | ||
| pub all_wifi_groups: Vec<WifiNetworkGroup>, | ||
| /// Saved Wi-Fi profiles keyed by SSID. |
There was a problem hiding this comment.
I don't yet know how I feel about breaking the current shape of AppletNetworkSummary...I am at work but will think on this more soon. Surely, there is a better way to approach this.
There was a problem hiding this comment.
I've been looking into this more and I've found a way to preserve the api by creating a similar structure for applet_summary as I used for wifi_groups. This would enable users to determine which they want and keep the same api bindings for both, but would require different function calls to receive hidden ap's in both cases. If there are any other ideas I'm happy to explore them as well, but if this sounds like a possible solution I can send it up for your review.
There was a problem hiding this comment.
Hmm yeah I was thinking something similar.
Happy to review these changes whenever you'd like to add them.
cachebag
added
api-surface
cachebag
changed the title
|
I'll make sure to include the issue number in the commits in the future and be clearer about the connections to the issue's in my PR's. I apologize for the inconvenience this has caused and will be more mindful in the future. |
2 participants
A minor refactor to address issue #495. I believe this will make hidden ap's available, and should now hide them by default. If we wanted to also handle the parsing of the hidden ap's names I can work on that next, however I believe that is out of the scope of this issue.
I addressed some of the examples to show that all_wifi_groups are available; however I did not add them to many examples that did not use them directly.