Merge lp:~andreagrandi/ubuntu-community-accomplishments/user-has-mugshot into lp:~ubuntu-community-accomplishments-reviewers/ubuntu-community-accomplishments/0.1
| Status: | Merged |
|---|---|
| Merged at revision: | 34 |
| Proposed branch: | lp:~andreagrandi/ubuntu-community-accomplishments/user-has-mugshot |
| Merge into: | lp:~ubuntu-community-accomplishments-reviewers/ubuntu-community-accomplishments/0.1 |
| Diff against target: |
65 lines (+56/-0) 2 files modified
accomplishments/ubuntu-community/user-has-mugshot.accomplishment (+18/-0) scripts/ubuntu-community/user-has-mugshot.py (+38/-0) |
| To merge this branch: | bzr merge lp:~andreagrandi/ubuntu-community-accomplishments/user-has-mugshot |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Jono Bacon | 2012-03-02 | Approve on 2012-03-05 | |
|
Review via email:
|
|||
Description of the Change
Add user-has-mugshot accomplishment script. This trophy is awarded when a user set an avatar image in the Launchpad website.
- 34. By Andrea Grandi on 2012-03-02
-
Fix some errors and debug code in user-has-mugshot script.
- 35. By Andrea Grandi on 2012-03-04
-
Add user-has-mugshot accomplishment. Some bugs fixed.
| Andrea Grandi (andreagrandi) wrote : | # |
> Thanks for the contribution, Andrea! This looks great. I think I am going to
> break this out into a set of Launchpad accomplishments for general things you
> can do in Launchpad, as this one is not specific to Ubuntu.
>
> In the meantime, could I ask you to be a bit more expansive in .accomplishment
> file and explain what a mugshot is, what are good mugshots (e.g. faces), what
> are bad ones (e.g. offensive images), and break the instructions in the
> 'steps' section into multiple steps - each step can be started on a new line
> with a tab before it.
>
> If you could fix up this I will then merge it.
>
> Thanks!
I've pushed the required changes into n°35 revision. Please let me know if it is ok!
Have a nice weekend :)
| Jono Bacon (jonobacon) wrote : | # |
Thanks for being so responsive!
I have merged this in. I just made two small changes:
* You had 'depends=' but it was empty so I set it to 'depends=
* I set the category to Launchpad, but as I said earlier, I might break this out into a different accomplishments set later. For now, it's own category works fine. :-)
Thanks, Andrea!

Thanks for the contribution, Andrea! This looks great. I think I am going to break this out into a set of Launchpad accomplishments for general things you can do in Launchpad, as this one is not specific to Ubuntu.
In the meantime, could I ask you to be a bit more expansive in .accomplishment file and explain what a mugshot is, what are good mugshots (e.g. faces), what are bad ones (e.g. offensive images), and break the instructions in the 'steps' section into multiple steps - each step can be started on a new line with a tab before it.
If you could fix up this I will then merge it.
Thanks!