#553 Patches welcome (dist-git PR merged)
Opened 6 years ago by churchyard. Modified a year ago

Badge description (like "You added a co-maintainer to a package. BFF!"):
You've merged a dist-git package Pull Request.

Help the badges team understand what this idea is all about. If this badge is
awarded for certain kinds of activities:

1) What are those activities?
- Merging a Pull Request in http://src.fedoraproject.org/ Pagure.

2) Who is doing them?
- packagers

3) Why are they doing them (is this a means to a different end?)
- to accept changes by other users

4) When do they do them (every day? once a year?)
- few times a year for a package is my educated guess

5) How do they do them (by talking in IRC? by running commands in the console?
by using a web interface?)
- by using a web interface (or API) at http://src.fedoraproject.org/

Lastly, do you have any ideas for artwork concepts?
- panda with thumbs up over a pile of paper?


Metadata Update from @riecatnor:
- Custom field artwork adjusted to None
- Custom field concept_review_passed adjusted to None
- Issue tagged with: artwork-needed, development

6 years ago

Metadata Update from @riecatnor:
- Custom field artwork adjusted to needed (was: None)

6 years ago

Metadata Update from @svitek:
- Issue assigned to svitek

6 years ago

@churchyard Hey, here is an artwork concept.

@svitek, great start! @mleonova and I talked about this in the meeting, and we think it could use a few improvements... The table in the bg is not really coming across as a table. Maybe you could try a version more like this https://badges.fedoraproject.org/badge/fedora-cz-author

Also, the proportions on the panda are looking a little off. The head is huge compared to the body and the stroke should be the same all over. Try making the head a little smaller, and make sure the stroke is the same on all.

I know you are working in the same office with @mleonova, so bug her to help you out :)

Hey, @svitek ! We discussed this one at the badges meeting today and agreed that we like the new concept much better!
Couple suggestions to make it perfect:
- tilt the paper a little bit - its straight up and down right now and will fit in more with badges style off balance
- the angle of the panda arm coming in is a bit weird with angle of the stamp. I'd suggest rotating it counter-clockwise slightly;
- outline on paper should be a slightly darker grey to pop off bg

Looking forward to seeing updated design!

@svitek checking in on this badge, any progress with the artwork? thanks

@mleonova Hey, fixed what you suggested.

@svitek great, thank you! The next step would be to ask @churchyard if he's happy with the design.

Metadata Update from @mleonova:
- Issue untagged with: artwork-needed
- Issue tagged with: artwork-approved

6 years ago

Metadata Update from @mleonova:
- Custom field artwork adjusted to approved (was: needed)

6 years ago

Metadata Update from @mleonova:
- Issue tagged with: ready to push

6 years ago

ok, setting artwork to approved and badge is ready to push.

BTW Before we push this. Juts a random thought (sorry for not bringing it sooner). Should this be a series instead? This is one things that can happen over and over again. When I proposed this badge idea, PRs were a new thing. Now it seems there are plenty of them. So we could have 1, 10, 50, 250 like with Koji.

I am +1 with the idea. I was planning to push this badge but I can skip for now. We can directly have the series.

Metadata Update from @churchyard:
- Issue untagged with: ready to push

5 years ago

@mleonova @churchyard @svitek -- thoughts on the numbering addition?

Metadata Update from @riecatnor:
- Issue tagged with: ready to push

5 years ago

ok, added tag ready to push back in

Metadata Update from @churchyard:
- Custom field has_description adjusted to on
- Custom field has_name adjusted to on
- Issue untagged with: ready to push

5 years ago

Metadata Update from @churchyard:
- Issue tagged with: needs-rules

5 years ago

I've created new tag needs-rules and applied it here, removing ready to push.

Issue tagged with: AW: needed

2 years ago

Metadata Update from @riecatnor:
- Issue untagged with: AW: needed

a year ago

Login to comment on this ticket.

Metadata
Attachments 7
Attached 6 years ago View Comment
Attached 6 years ago View Comment
Attached 6 years ago View Comment
Attached 6 years ago View Comment
Attached 6 years ago View Comment
Attached 6 years ago View Comment
Attached 5 years ago View Comment