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
Branch
fix-flag
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 615
Build 615: arc lint + arc unit

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.