#543 implemend badges widget as per mockup
Merged by abompard. Opened by ryanlerch.
ryanlerch/fedora-hubs badges-widget  into  develop

Download 543.patch

we don't habe the "rarest" badge info from the API
but everything else is there and works.

badges-widget.png

fixes #435

rebased onto b5a63530d22c53a3c035043e05c0d32506fe8806

Do I really want this widget on my stream page? Am I that obnoxious? ;-p

Hmm, this seems wrong. Was it for testing only?

In case of server error, I think the json() call fails and that's why it was put in the try-except below (it's the ValueError case).

leftovers?

It feels like there should be more links to badges.fp.o : the blue "badges earned" number seems like a good candidate, it makes me want to click on it :-)

It feels like there should be more links to badges.fp.o : the blue "badges earned" number seems like a good candidate, it makes me want to click on it :-)

Link added! I meant to add this one, but totally forgot previously!

rebased onto 41a954a5f483a0665af94beae2322ec2a186f022

Have also removed the cache invalidation bit -- that was a vestige from testing the widget! Also removed the two commented out lines.

The .json call is also now in the try-except block.

Also, i supposed some people might want to keep up on their rating every day (i know people at the top of the list take it very seriously :) ) -- that is why i added it as an option to the stream page. :D

Beautiful, thanks! :-)

Pull-Request has been merged by abompard

Metadata