I think we should deal with this stuff first:
- T9010 (may require migrations)
- T10798 (limit stuff only, I don't really care about order much -- badges currently won't scale on pages that use `needRecipients()` and/or aren't paginated if a badge is awarded to a large number of recipients)
- T10698
- I can repeatedly award a badge to the same recipients with no UI feedback:
{F2930527}
- The timeline view does PHP-side filtering again, though.
- Since the timeline view doesn't even use needBadges on the People query, can we get rid of it so this feature isn't totally tied to the core?
- The "Qualities" constraint in `badges.search` accepts numeric strings like "102". Maybe just drop this constraint from Conduit? I don't really love how badge qualities work internally, and if we hide them we don't have to fix this yet.
{F2931053}
- Mail sends with subject lines like "[Badge] [Updated] Badge 6", and does not list the name of the badge.
- Mail includes "BADGE DESCRIPTION" section in every message about a badge with a description. Particularly, edits to the badge description have the edit and then the description. The description should only be included in the initial creation mail, like Maniphest:
{F2931241}