Merge lp:~unity-api-team/indicator-network/hotspot-cellular-enabled-properties into lp:indicator-network/15.10
Proposed by
Pete Woods
Status: | Merged |
---|---|
Approved by: | Pete Woods |
Approved revision: | 518 |
Merged at revision: | 525 |
Proposed branch: | lp:~unity-api-team/indicator-network/hotspot-cellular-enabled-properties |
Merge into: | lp:indicator-network/15.10 |
Diff against target: |
453 lines (+192/-12) 13 files modified
data/com.ubuntu.connectivity1.NetworkingStatus.xml (+3/-0) debian/changelog (+7/-0) debian/control (+1/-3) src/connectivity-api/connectivity-qt/connectivityqt/connectivity.cpp (+18/-0) src/connectivity-api/connectivity-qt/connectivityqt/connectivity.h (+10/-0) src/indicator/connectivity-service/connectivity-service.cpp (+29/-0) src/indicator/connectivity-service/connectivity-service.h (+6/-0) src/indicator/nmofono/hotspot-manager.cpp (+27/-0) src/indicator/nmofono/hotspot-manager.h (+8/-0) src/indicator/nmofono/manager-impl.cpp (+54/-9) src/indicator/nmofono/manager-impl.h (+4/-0) src/indicator/nmofono/manager.h (+10/-0) tests/integration/test-connectivity-api.cpp (+15/-0) |
To merge this branch: | bzr merge lp:~unity-api-team/indicator-network/hotspot-cellular-enabled-properties |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jonas G. Drange (community) | Needs Resubmitting | ||
Indicator Applet Developers | Pending | ||
Review via email: mp+268775@code.launchpad.net |
Commit message
Add properties to determine if modems and hotspot functionality are available.
Description of the change
Add properties to determine if modems and hotspot functionality are available.
To post a comment you must log in.
Does this scale at all? We should be making determinations about what device can have hotspot based on its capabilities—not the device name (are there no flo devices with modems, e.g.?)
I think this also makes the API ambiguous: we now have hotspotEnabled, hotspotSwitchEn abled, hotspotAvailable and hotspotStored. To be blunt, we'd need a logical matrix to determine what to show as enabled/ disabled/ populated/ N/A in the UI.
For mako, we should try to fix the actual bugs that are causing the hotspot to fail, and for flo wee need enumerated capabilities from somewhere (SDK) in order to hide the hotspot (and cellular, phone) entry.