Page MenuHomePhabricator

Remove support for the "TYPE_AUTH_WILLLOGIN" event
ClosedPublic

Authored by epriestley on Dec 18 2018, 7:38 PM.
Tags
None
Referenced Files
F14493383: D19904.diff
Fri, Jan 3, 2:12 AM
F14492972: D19904.diff
Thu, Jan 2, 11:39 PM
F14491573: D19904.diff
Thu, Jan 2, 8:47 AM
Unknown Object (File)
Tue, Dec 10, 10:32 PM
Unknown Object (File)
Thu, Dec 5, 6:25 AM
Unknown Object (File)
Dec 3 2024, 10:47 AM
Unknown Object (File)
Nov 27 2024, 9:19 AM
Unknown Object (File)
Nov 25 2024, 1:43 AM
Subscribers
None

Details

Summary

Depends on D19903. Ref T13222. This was a Facebook-specific thing from D6202 that I believe no other install ever used, and I'm generally trying to move away from the old "event" system (the more modern modular/engine patterns generally replace it).

Just drop support for this. Since the constant is being removed, anything that's actually using it should break in an obvious way, and I'll note this in the changelog.

There's no explicit replacement but I don't think this hook is useful for anything except "being Facebook in 2013".

Test Plan
  • Grepped for TYPE_AUTH_WILLLOGIN.
  • Logged in.

Diff Detail

Repository
rP Phabricator
Branch
mfa22
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21369
Build 29091: Run Core Tests
Build 29090: arc lint + arc unit