Merge lp://qastaging/~knitzsche/scope-aggregator/bug-1602141-only-show-help-for-installed into lp://qastaging/scope-aggregator
Proposed by
Kyle Nitzsche
Status: | Merged |
---|---|
Approved by: | Kyle Nitzsche |
Approved revision: | 174 |
Merged at revision: | 169 |
Proposed branch: | lp://qastaging/~knitzsche/scope-aggregator/bug-1602141-only-show-help-for-installed |
Merge into: | lp://qastaging/scope-aggregator |
Diff against target: |
86 lines (+36/-4) 3 files modified
CMakeLists.txt (+1/-1) data/hints.json (+1/-0) src/utils.cpp (+34/-3) |
To merge this branch: | bzr merge lp://qastaging/~knitzsche/scope-aggregator/bug-1602141-only-show-help-for-installed |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gary.Wang | Needs Fixing | ||
Zhang Enwei (community) | Approve | ||
Penk Chen | Pending | ||
Review via email:
|
Description of the change
do not display quick start help result for any click child scope that is not installed.
To post a comment you must log in.
Hi Kyle, providername and it is generated by online- account- hooks2. bazaar. launchpad. net/~facebook- scope-team/ facebook- scope/trunk/ revision/ 144#src/ query.cpp /bazaar. launchpad. net/~hanloon- team/hanloon/ unity-scope- fitbit/ revision/ 48#src/ fitbit. go /developer. ubuntu. com/en/ phone/platform/ guides/ online- accounts- developer- guide/(there are errors in this page)
For scopes that are using new online account settings, the service name is not the scope id. It is scopeid_
Please refer to
http://
https:/
https:/