What the badge should be granted for: package review Badge description (like "You succesfully approved a package"): Anything else we need to know: this should scale 1, 10, 50, 100 packages
Name "Package reviewed"
I think this requires BZ integration (as with several other proposals) as that's the only way 'review passed' is indicated. This doesn't mean we're rejecting the idea, just that it has to wait till that gets done!
Thanks for the idea!
Huge +1 for this one.
FWIW: Huge +1 also from me
Yeah, this would be amazing, but we're still waiting on bugzilla+fedmsg integration.
+1
Copy paste from another ticket, closed as duplicate: Replying to pnemade:
I have seen similar ticket sometime ago, can't find it now but this requires fedmsg bugzilla integration which I think is not yet implemented.
Bodhi could check for the valid review in Bugzilla, when the package update was successfully sent for the first time.
If you are saying to award badge for number of package reviews done by a reviewer then I am not sure how can bodhi help here. Bodhi is for package maintainers not for package reviewers.
Reviewer could be read from review assignment in Bugzilla.
This should be now possible: https://fedora-fedmsg.readthedocs.org/en/latest/topics.html#bugzilla-bug-update
Current discussion thread on devel mailing list about review statistics. https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/I4D77CXDEWEPQCQLEVIRIDQJH2LAEGGH/
Replying to [comment:11 raphgro]:
Yes, if you would like, I could additionally update the script to play nicely with Badges.
Replying to [comment:12 clime]:
Replying to [comment:11 raphgro]: Current discussion thread on devel mailing list about review statistics. https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/I4D77CXDEWEPQCQLEVIRIDQJH2LAEGGH/ Yes, if you would like, I could additionally update the script to play nicely with Badges.
That would be really cool with an automated script for doing the awards. Fedmsg can not tell about reviews done in past, rhbz integration is still rudimental. The script should delay for some time period to be somehow tolerant against errors cause people sometimes set the wrong flags in rhbz and change back and forth, like with fedora-review.
Is this badge possible? It looks like it might be.
Artwork concept: Panda with a uniform on and a package on it's shirt with a check mark Mash up of these badges: panda: https://badges.fedoraproject.org/badge/famsco-member package: https://badges.fedoraproject.org/badge/building-the-outer-ring-copr-build-i Check mark: https://badges.fedoraproject.org/badge/if-you-build-it...-koji-success-i
Metadata Update from @riecatnor: - Custom field artwork adjusted to has_concept - Custom field concept_review_passed adjusted to passed (was: 0) - Custom field has_complete_yaml reset (from Partial) - Custom field has_description adjusted to on (was: 1) - Custom field has_name adjusted to on (was: 1) - Custom field needs_manual_award reset (from 0) - Custom field triaged adjusted to on (was: 1) - Issue close_status updated to: None
well, the information is in fedmsg...
See https://apps.fedoraproject.org/datagrepper/id?id=2017-b5959f7d-912a-4abb-87d1-c255c1bced88&is_raw=true&size=extra-large
msg.event.changes[0].added == fedora-review+
It does not holds any information on the user FAS name though (just BZ email).
big +1 on this one though :)
Metadata Update from @riecatnor: - Custom field has_complete_yaml reset (from false) - Custom field needs_manual_award reset (from false) - Issue tagged with: artwork-easy
Metadata Update from @riecatnor: - Custom field has_complete_yaml reset (from false) - Custom field needs_manual_award reset (from false)
Metadata Update from @riecatnor: - Custom field has_complete_yaml reset (from false) - Custom field needs_manual_award reset (from false) - Issue tagged with: artwork-needed, quality
I want this. But with a larger scale maybe: 100, 250, 500.
@riecatnor Any chance you have some time to work on this?
@eclipseo we can definitely work on this one, but before we go for it, could you please check if this is technically possible? That would be great =) Thank you!
Metadata Update from @mleonova: - Custom field has_complete_yaml reset (from false) - Custom field needs_manual_award reset (from false)
This is technically possible.
Metadata Update from @churchyard: - Custom field has_complete_yaml reset (from false) - Custom field needs_manual_award reset (from false)
Metadata Update from @mleonova: - Issue assigned to mleonova
Here's a proposal for this badge based on @riecatnor's concept: <img alt="101-package-reviewer-1.svg" src="/fedora-badges/issue/raw/files/08171def376e01918c82fb04d5eec540a3034974e5555489eade4981bf1a675e-101-package-reviewer-1.svg" /> <img alt="101-package-reviewer-1.png" src="/fedora-badges/issue/raw/files/3d68ad6c2d3d63769fdb468af1d1468a3abac4f56e3321608ac088ea970d9b5b-101-package-reviewer-1.png" />
It looks pretty neat! Thank you!
Love it! Should it be development over quality? (Not sure.)
Very nice.
What do you think? I am even more not sure :)
I like how it looks in green. And doing a review is sort of about quality assurance. So let's keep it green.
@eclipseo @churchyard so should we do 1, 10, 50, 100 or 100, 250, 500? What looks more realistic?
1, 10, 50, 100 is good. Not many people get to 100 reviews, and those who do, already have many other badges ;) I think it's most important to have a few badges at the low end, to allow relative beginners to have positive feedback. So I'd even make it 1-5-15-50.
Thank you for your work here. It is great to see this is going somewhere.
I would make it a bit harder to get the badges, since we don't want to encourage people to do reviews just like flipping a coin. That'd make my preference to 1,10,50,100.
I'm partial to higher scale because I've done so many reviews (near 1800) but I guess 1, 10, 50, 100 is more realistic to encourage participation of newcomers.
10 reviews:
<img alt="101-package-reviewer-10.png" src="/fedora-badges/issue/raw/files/2da5910b711d70d3bcadac29e9a23ba5003cb0f5b930cceb6b326bf39432629d-101-package-reviewer-10.png" /><img alt="101-package-reviewer-10.svg" src="/fedora-badges/issue/raw/files/10a06b5f19942bbd0b262e59705f68ac411fd0245438ebb6f0fe5f336d9b087c-101-package-reviewer-10.svg" />
50 reviews:
<img alt="101-package-reviewer-50.svg" src="/fedora-badges/issue/raw/files/0ba17705f2211bcd3340b6df8b6e2c9a56846adb51c52ef09ab0a829276d8366-101-package-reviewer-50.svg" /><img alt="101-package-reviewer-50.png" src="/fedora-badges/issue/raw/files/a2df266fd592e671aeab02c5c7ff20f13c6719ee67cc4429a20148867cabe269-101-package-reviewer-50.png" />
100 reviews:
<img alt="101-package-reviewer-100.svg" src="/fedora-badges/issue/raw/files/be62683aada9177d0130a6138ca9588cc5620de6d2c8270565ede24d9e6562d7-101-package-reviewer-100.svg" /><img alt="101-package-reviewer-100.png" src="/fedora-badges/issue/raw/files/5e78a2946497318dcc38c23c14046097ec60b1105398e93b9ea89157edaef857-101-package-reviewer-100.png" />
@eclipseo wow! Impressive!
these are lookin' good @mleonova - setting to approved :)
Metadata Update from @riecatnor: - Custom field artwork adjusted to approved (was: has_concept) - Custom field has_complete_yaml reset (from false) - Custom field needs_manual_award reset (from false)
Metadata Update from @riecatnor: - Custom field has_complete_yaml reset (from false) - Custom field needs_manual_award reset (from false) - Issue untagged with: artwork-needed - Issue tagged with: artwork-approved, ready to push
Can we create the rules file and push this on the Flock workshop?
Flock 2019 badge issue triage, closing old tickets. This one looks so close to ready tho! Any chance someone can take a look at this one and get these pushed? Thanks!!
And what about the rules file?
Metadata Update from @riecatnor: - Custom field has_complete_yaml reset (from false) - Custom field needs_manual_award reset (from false) - Issue untagged with: difficulty - easy
Metadata Update from @smeragoel: - Custom field has_complete_yaml reset (from false) - Custom field needs_manual_award reset (from false) - Issue untagged with: ready to push - Issue tagged with: needs-rules
Issue tagged with: AW: needed
Metadata Update from @riecatnor: - Custom field has_complete_yaml reset (from false) - Custom field needs_manual_award reset (from false) - Issue untagged with: AW: needed
Log in to comment on this ticket.