Allow people to award and remove badges via conduit, but not from the standard badges form.
- Maniphest Tasks
- T12270: Unbeta Badges
- rPfa569c35d30e: Add award and revoke conduit calls to Badges
Build a generator and generate awards. Didn't test the revoke yet.
I think we should probably just hold this for now since we don't really have use cases yet.
If we build "award reason" in the future (where an awarder can explain why they awarded the badge, which seems nice to have and like something we'll probbbbably build eventually?) then a separate badge.award.edit probably makes more sense (so you can provide a reason, edit it, etc). Or if we let badges be awarded multiple times ("1st Place Finish in Company Sack Race" for 2018, 2019, and 2021), since then revocation would need to specify which award was being revoked.
If we don't build that stuff then something like this makes more sense, but it's hard/impossible to extend to those use cases without compatibility breaks if we do build those features.
I think that's a reasonable-enough use case. We could do the lipsum thing by using a raw Editor, but I think using Conduit is cleaner. One inline about naming/consistency. If we do end up making a small break here in the future that's not the end of the world.
Minor, but prefer award and revoke for consistency -- no other keys have underscores.
Yeah this seemed minimal to maintain and even if we had badge.award.edit, we could still keep this around. I do like the long-term idea of Badges + Facts for automatic awards. I tried to futz with Herald, but it felt forced.