1165514 - [gh-pages] on http://mozilla.github.io/mozregression/, show the number of gui releases downloaded

Status: RESOLVED FIXED

(Testing :: mozregression, defect)

Description

[Julien Pagès (:parkouss)]

It would be good to show the number of releases downloaded for each os (Linux64 and Windows for now) in the main page of the mozregression web site: http://mozilla.github.io/mozregression/. This way we will have an easy way to see the actual number of downloads for the tool!

To do that, we can query the github api: https://api.github.com/repos/mozilla/mozregression/releases. I suggest that we simply sum the numbers of downloaded release, for each os.

Some knowledge of html, javascript and jekyll is required here, though I can help if needed. :)

The code lives on the gh-pages branch of mozregression, and the README can help to set up things.


After that, doing the same thing to count downloads on pypi could be interesting too - maybe just show a badge, or query pypi in the same way. This can be done in another bug.

Comment 1

[MikeLing]

Hi~ I would like to work on this bug if it's ok :)

I have html and javascript but little about jekyll. However I can learn how to use it(it seems like a tools make your website can get data from github). 

This issue looks like need I get the "download_count" of each os (.exe and .zip or .tar) from this page (https://api.github.com/repos/mozilla/mozregression/releases)and reflect it on mozregression web site.

Comment 2

[Julien Pagès (:parkouss)]

Yup, you're welcome to work on this! What you said is right, this is what I would like to do here.

So you have to checkout the gh-pages branch of mozregression, then just follow this to get started:

https://github.com/mozilla/mozregression/blob/gh-pages/README.md

After that, try to understand how that work by looking at the jekyll documentation:

http://jekyllrb.com/

And ask me if you need to!

@wlach
Do you like the idea here ? And do you have another idea to where to put the download count information ?

Comment 3

[William Lachance (:wlach)]

Sure, this is fine by me. ;)

Comment 4

[Julien Pagès (:parkouss)]

Hm MikeLing I'm not sure I understoog well here. Are you still working on this bug ? Else I will unassign you, so somebody else can look at this.

Comment 5

[Julien Pagès (:parkouss)]

MikeLing is working on other stuff for now, so let's see if someone else may be interested to work on this bug.

Comment 6

[joridoss]

Assign to me. :D

Comment 7

[Julien Pagès (:parkouss)]

Thanks joridoss this looks good ! (https://github.com/mozilla/mozregression/pull/231)

There are some minor changes I would like to see. Instead of:

> Downloads:
>
> gui-0.2
>    linux: 4
>    windows: 31
>
> gui-pre-release
>    linux: 2
>    windows: 5

Would it be possible to see:

> Number of downloads for the gui version:
>
> linux: 6
> windows: 36

I don't think we need the details for each release (at least for a first look at the page, you could add a link "details" for example that could expand to detailed information for each release if you're interested!).

Also I would prefer to put this block at the end of the page (after the "Recent news section"), so the page layout won't change much once the data is loaded.


Last thing, when submitting patches for mozregression you should ask for a review here in bugzilla (see https://wiki.mozilla.org/Auto-tools/Projects/Mozregression#Submitting_patches). You have to do this by adding an attachment on this bug, put the url of the github PR in the main field (see https://globau.wordpress.com/2013/10/21/github-pull-requests-and-bugzilla/), choose "?" in the review flag and put my email address to choose me as a reviewer.

Thanks again for tackling this bug! Please ask if you have any question. :)

Comment 8

[joridoss]

Oks, I will make those changes this week.

Comment 9

[Julien Pagès (:parkouss)]

joridoss, are you still interested/do you have time to work on this ?

Comment 10

[joridoss]

This and the next month I have a lot of things to do in my Job, so, anyone could clone my branch and make the changes

Comment 11

[Julien Pagès (:parkouss)]

(In reply to joridoss from comment #10)
> This and the next month I have a lot of things to do in my Job, so, anyone
> could clone my branch and make the changes

Right, thanks for the quick reply! And also for the work, we're almost done thanks to you. :)

Comment 12

[Giovanny Gongora [:gioyik]]

Attachment: Github PR

Comment 13

[Giovanny Gongora [:gioyik]]

Hi Julien,

I submitted a patch based on joridoss PR, but I made it a bit easier. Let me not if is good to land or needs something else.

Thanks

Comment 14

[Julien Pagès (:parkouss)]

Comment on attachment 8637682 [details] [review]
Github PR

Hey Giovanny, thanks!

This is good, the main thing is that I would like a total of the downloads by platform (not specific to one release). I thought I said that in comment 7, but it is not obvious at all! Sorry for that. :)

I noted some comment on the PR. Removing the review flag for now, please set it back when you're done.

Comment 15

[Julien Pagès (:parkouss)]

Givanny, do you still want to work on this bug ? Else I'm going to reset the assignee, see if someone wants to take care of it.

Comment 16

[joridoss]

Attachment: Pull request with the right info.

I changed the code to show all the OS type info, and I put the code in the bottom.

Comment 17

[Julien Pagès (:parkouss)]

(In reply to joridoss from comment #16)
> Created attachment 8677827 [details] [review]
> Pull request with the right info.
> 
> I changed the code to show all the OS type info, and I put the code in the
> bottom.

This looks perfectly fine by me! Thanks for doing this. :)

If you know how to do this, please rebase your commits to have only one for this patch (else I can do it).
Also if you could ask for a review so I can r+ and land. (you have to select "?" in the r field, with my email address).

Glad to see this!

Comment 18

[joridoss]

Can u rebase for me?

Comment 19

[Julien Pagès (:parkouss)]

Yup, no problem - will do before landing. Please just ask for a review then.

Comment 20

[Julien Pagès (:parkouss)]

Well, no real need for a review flag anyway, I'm totally happy with the changes. :)

Rebased and landed in https://github.com/mozilla/mozregression/commit/b2db5044fd4ab2503bcf107f4f9385c4a58776c2.

Thanks Joridoss!