Merge lp:~philipballew/ubuntu-community-accomplishments/ubuntu-youth-council into lp:ubuntu-community-accomplishments

Proposed by philipballew
Status: Merged
Merged at revision: 247
Proposed branch: lp:~philipballew/ubuntu-community-accomplishments/ubuntu-youth-council
Merge into: lp:ubuntu-community-accomplishments
Diff against target: 64 lines (+48/-0)
3 files modified
accomplishments/ubuntu-community/en/governance/ubuntu-youth-council.accomplishment (+15/-0)
scripts/ubuntu-community/governance/ubuntu-youth-council.py (+28/-0)
tests/ubuntu-community/governance/ubuntu-youth-council (+5/-0)
To merge this branch: bzr merge lp:~philipballew/ubuntu-community-accomplishments/ubuntu-youth-council
Reviewer Review Type Date Requested Status
Matt Fischer Approve
Jono Bacon Needs Fixing
Rafał Cieślak Needs Fixing
Review via email: mp+119073@code.launchpad.net

Description of the change

I changed some Ubuntu Youth council stuff for you!

To post a comment you must log in.
Revision history for this message
Rafał Cieślak (rafalcieslak256) wrote :

Awesome work, huge thanks.
There is one more tiny thing that will need fixing: The accomplishment icon has white background (instead of transparent) which causes it to look oddly in notification bubbles and viewer's trophy list. Can you please fix it? ;-)

review: Needs Fixing
218. By philipballew

added better ubuntu-youth image

219. By philipballew

added better ubuntu-youth image and changed accomplishment file to match

Revision history for this message
Jono Bacon (jonobacon) wrote :

Is this now ready for review again, Phillip?

review: Needs Information
Revision history for this message
philipballew (philipballew) wrote :

Yes. I have made the corrections I was instructed to to. If there is
anything else needed feel free to let me know and I'll get on making
those as well.

On Mon, Aug 13, 2012 at 7:19 PM, Jono Bacon <email address hidden> wrote:
> Review: Needs Information
>
> Is this now ready for review again, Phillip?
> --
> https://code.launchpad.net/~philipballew/ubuntu-community-accomplishments/ubuntu-youth-council/+merge/119073
> You are the owner of lp:~philipballew/ubuntu-community-accomplishments/ubuntu-youth-council.

Revision history for this message
Jono Bacon (jonobacon) wrote :

Thanks, Philip for your contribution!

Just a few small changes needed:

 * Please flesh out the summary - explain what the youth team is, its goals, who can join etc. Always assume the reader has no idea about the team in question. :-)
 * Could you flesh out to the steps to include details of how someone would apply to be on the council. See the Community Council accom as an example of the steps fleshed out.
 * Please set the category to Governance.
 * There are a few typos in there, please make sure you run the text through a spell checker.

Other than these small changes, this is ready to go! :-)

review: Needs Fixing
Revision history for this message
Matt Fischer (mfisch) wrote :

Any updates on this? I can merge it in once it's all ready.

220. By philipballew

fixed

Revision history for this message
Matt Fischer (mfisch) wrote :

I think it looks good. I'm going to switch the depends to being a member of the ubuntu-youth team since it's a closer pre-req. Next time you should probably run spell check on this to catch some of the typos. Thanks for contributing it!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'accomplishments/ubuntu-community/en/governance/ubuntu-youth-council.accomplishment'
2--- accomplishments/ubuntu-community/en/governance/ubuntu-youth-council.accomplishment 1970-01-01 00:00:00 +0000
3+++ accomplishments/ubuntu-community/en/governance/ubuntu-youth-council.accomplishment 2012-12-23 03:00:29 +0000
4@@ -0,0 +1,15 @@
5+[accomplishment]
6+title=Approved Ubuntu Youth Council Member
7+description=You are an Approved Ubuntu Youth Council Member
8+collection=ubuntu-community
9+category=Governance
10+needs-signing=true
11+depends=ubuntu-community/registered-on-launchpad
12+icon=ubuntu-youth.png
13+needs-information=launchpad-email
14+summary = Someone who is part of the Ubuntu Youth Council is somebody who has spent a great deal of time helping to contribute to the Ubuntu Youth project and was able to win an elcetion from other members of Ubuntu Youth that shows they had the acility to help lead Ubuntu Youth. all indivuidals of the Ubuntu Youth Council will have to attend regualar meetengs and be able to activitly partisibate in the decisions regarding the Ubuntu Youth team.
15+steps: Join the Ubuntu Youth team and contribute untill the next Ubuntu Youth Council election. When a call for namanations come out, nominate yourself and see if you win
16+tips: Be sure to be positive and get to know everyone in the Ubuntu youth group.
17+links: https://launchpad.net/~ubuntu-youth-council
18+help = #ubuntu-youth-council on Freenode
19+author = Philip Ballew <philipballew@ubuntu.com>
20
21=== added file 'accomplishments/ubuntu-community/trophyimages/ubuntu-youth.png'
22Binary files accomplishments/ubuntu-community/trophyimages/ubuntu-youth.png 1970-01-01 00:00:00 +0000 and accomplishments/ubuntu-community/trophyimages/ubuntu-youth.png 2012-12-23 03:00:29 +0000 differ
23=== added file 'scripts/ubuntu-community/governance/ubuntu-youth-council.py'
24--- scripts/ubuntu-community/governance/ubuntu-youth-council.py 1970-01-01 00:00:00 +0000
25+++ scripts/ubuntu-community/governance/ubuntu-youth-council.py 2012-12-23 03:00:29 +0000
26@@ -0,0 +1,28 @@
27+#!/usr/bin/python
28+import traceback, sys
29+
30+from accomplishments.daemon import dbusapi
31+# Add scripts/lib/ to the PYTHONPATH
32+import os
33+sys.path.append(os.path.abspath(os.path.join(os.path.dirname(__file__), '..', 'lib')))
34+from helpers import Launchpad
35+
36+try:
37+ api = dbusapi.Accomplishments()
38+ f = api.get_extra_information("ubuntu-community", "launchpad-email")
39+ if bool(f[0]["launchpad-email"]) == False:
40+ sys.exit(4)
41+ else:
42+ email = f[0]["launchpad-email"]
43+
44+ me = Launchpad.fetch(email)
45+ if "ubuntu-youth-council" in me.super_teams:
46+ sys.exit(0)
47+ else:
48+ sys.exit(1)
49+
50+except SystemExit, e:
51+ sys.exit(e.code)
52+except:
53+ traceback.print_exc()
54+ sys.exit(2)
55
56=== added file 'tests/ubuntu-community/governance/ubuntu-youth-council'
57--- tests/ubuntu-community/governance/ubuntu-youth-council 1970-01-01 00:00:00 +0000
58+++ tests/ubuntu-community/governance/ubuntu-youth-council 2012-12-23 03:00:29 +0000
59@@ -0,0 +1,5 @@
60+[success]
61+launchpad-email = philipballew@ubuntu.com
62+
63+[failure]
64+launchpad-email = fail@fail.com

Subscribers

People subscribed via source and target branches

to all changes: