548361 - Minor adjustments of API 1.5

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: API, defect, P2)

Description

[Wil Clouser [:clouserw]]

Currently API1.5 shows:

<num_reviews>1569</num_reviews><reviews>/z/en-US/firefox/addon/1865/reviews?src=api</reviews>

This should be combined into:

<reviews num="20">URL</reviews>

Also on the install attribute:

size="300kb"

should become:

size="300" units="kb"

An example URL:
https://preview.addons.mozilla.org/z/en-US/firefox/api/1.5/addon/1865

Comment 1

[Pratibha]

https://preview.addons.mozilla.org/z/en-US/firefox/api/1.5/addon/1569

Reviews section: Fixed

<reviews num="4">
https://preview.addons.mozilla.org/z/en-US/firefox/addon/1569/reviews/?src=api
</reviews>

Install: Do not see the attribute
Units="kb"

<install hash="sha256:6e8d5ba0626ec8c78bb38c2820744b217b32cb74f827488d4da9bf27c2dfe578" os="ALL" size="5120">

    https://preview.addons.mozilla.org/downloads/file/6756/cookie_crumbler-1.0-fx.xpi?src=api
  
</install>

Comment 2

[Jeff Balogh (:jbalogh)]

Pratibha: If this is fixed, please change the status from Resolved to Verified.

Comment 3

[krupa raj[:krupa]]

(In reply to comment #0)
> Currently API1.5 shows:
> 
> <num_reviews>1569</num_reviews><reviews>/z/en-US/firefox/addon/1865/reviews?src=api</reviews>
> 
> This should be combined into:
> 
> <reviews num="20">URL</reviews>
> 
> Also on the install attribute:
> 
> size="300kb"
> 
> should become:
> 
> size="300" units="kb"
> 
> An example URL:
> https://preview.addons.mozilla.org/z/en-US/firefox/api/1.5/addon/1865

install attribute currently has the following info:

<install hash="sha256:e691e8230286883726203768bb1d22c9d19ded7165523a298edddc1cb25e886c" os="ALL" size="307200">

    https://preview.addons.mozilla.org/downloads/file/74835/adblock_plus-1.1.3-fx+sm+tb+fn.xpi?src=api
  
</install>


I do not see "units" specified under install attribute


reopening this bug.

Comment 4

[krupa raj[:krupa]]

Attachment: units missing

Comment 5

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

We took out units in favor of multiplying by 1024 to get bytes:

https://bugzilla.mozilla.org/show_bug.cgi?id=543559