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

Revision history for this message
Gary.Wang (gary-wzl77) wrote :

Hi Enwei
   Sorry for the late reply.
   Today I added this feature in bollywood aggregator to verify it, here is the child_json.file
   https://bazaar.launchpad.net/~hanloon-team/hanloon/bollywood_opt_keyword/revision/24

   However, I can see some scope installation recommendation in scope that I don't list in keyword_scope_ids. Please have a check the snapshot below
   https://drive.google.com/open?id=0B2H9ECPSSfqIS2UxX3FzYU9peW8

   Also please see my inline comment below. I will merge my branch into trunk later after this problem is fixed as this one is prerequisite branch.
  Thanks.

   Here is my image info
  current build number: 34
device name: krillin
channel: ubuntu-touch/stable/bq-aquaris.en
last update: 2016-08-01 15:23:14
version version: 34
version ubuntu: 20160708
version tag: OTA-12
version device: 20160606-ab415b2
version custom: 20160701-981-38-14

review: Needs Fixing

« Back to merge proposal