Page MenuHomePhabricator

Remove obsolete Badges edge types
ClosedPublic

Authored by epriestley on Mar 24 2017, 9:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 22, 10:24 AM
Unknown Object (File)
Fri, Dec 13, 6:18 AM
Unknown Object (File)
Sun, Dec 8, 2:45 PM
Unknown Object (File)
Thu, Dec 5, 7:58 AM
Unknown Object (File)
Wed, Dec 4, 12:15 AM
Unknown Object (File)
Fri, Nov 29, 8:30 PM
Unknown Object (File)
Mon, Nov 25, 6:08 PM
Unknown Object (File)
Nov 22 2024, 11:23 AM
Subscribers
None

Details

Summary

Ref T12270. These no longer have any callsites.

Test Plan

Used grep to search for each edge class constant, found no hits.

Diff Detail

Repository
rP Phabricator
Branch
badge2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16147
Build 21432: Run Core Tests
Build 21431: arc lint + arc unit

Event Timeline

would this affect old migrations to Awards?

  • Wow! Also run arc liberate!

No, they use the constants explicitly, anticipating this change.

It will cause old award transactions to render "epriestley edited edges." or similar instead of "epriestley awarded this badge to chad.", but that's the only negative effect and I'm not planning to fix it since the app was beta when those awards happened.

(They also all happened more than a year ago.)

Or, at least, happened on a version of the codebase which is more than a year old

This revision is now accepted and ready to land.Mar 24 2017, 9:09 PM
This revision was automatically updated to reflect the committed changes.