Page MenuHomePhabricator

Fix flag admin url resolving
ClosedPublic

Authored by Info-Screen on Apr 9 2019, 6:43 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 27, 4:54 AM
Unknown Object (File)
Tue, Sep 27, 4:54 AM
Unknown Object (File)
Tue, Sep 27, 4:54 AM
Unknown Object (File)
Mon, Sep 26, 12:20 PM
Unknown Object (File)
Sat, Sep 24, 10:52 PM
Unknown Object (File)
Sat, Sep 24, 7:52 PM
Unknown Object (File)
Fri, Sep 23, 5:01 AM
Unknown Object (File)
Fri, Sep 23, 3:54 AM
Subscribers
None

Details

Summary

The url resolving for the flag admin still belived we had different post admins for
each area. This is no longer the case.
Also adding a test, that checks for this issue.
Side note: I also saw some debug output, the test admin still head. Removed that.

Test Plan
  • Flag a post and check flag admin.
  • Click on link to post admin

Diff Detail

Repository
rAPI Server API
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Apr 9 2019, 1:54 PM
This revision was automatically updated to reflect the committed changes.