Code review comment for lp://qastaging/~zhangew401/scope-aggregator/fix-lp-1524597

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

Hi Enwei,

Just for clarity, can you please use this in the README instead of the propsed text?

This key is only used on the special clickstore scope declared with "id": "com.canonical.scopes.clickstore".

This clickstore scope displays scopes that are not installed and allows the user to install them.

By default, the clickstore scope (when declared) displays all child scopes declared in the child scopes.jon file by scope ID that are not currently installed.

This key allows the clickstore scope to also display scopes that are aggregated by keyword and that are not installed (in addition to scopes declared by ID and not installed, if any).

Example:

(show your USUAGE EXAMPLE json)

Example with no keyword_sources:

        [...]
        "scope":
        {
            "_category_title": "Install More Sources:",
            "id": "com.canonical.scopes.clickstore",
            "link_to_child": "true",
            "local_id": "com.canonical.scopes.clickstore",
            "department": "dept-movies",
            "cardinality": 10,
        }
        [...]

« Back to merge proposal