Merge lp:~andreagrandi/ubuntu-community-accomplishments/user-has-mugshot into lp:~ubuntu-community-accomplishments-reviewers/ubuntu-community-accomplishments/0.1
Proposed by
Andrea Grandi
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 | Approve | ||
Review via email: mp+95690@code.launchpad.net |
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.
To post a comment you must log in.
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!