Differential D240

Fix flag admin url resolving
ClosedPublic
Actions

Authored by Info-Screen on Apr 9 2019, 6:43 AM.

Tags

None

Subscribers

None

Details

Reviewers

Commits

rAPI1ebf72a0269c: Fix flag admin url resolving

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

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Info-Screen created this revision.Apr 9 2019, 6:43 AM

Harbormaster completed remote builds in B615: Diff 833.Apr 9 2019, 6:43 AM

WyldBot accepted this revision.Apr 9 2019, 1:54 PM

This revision is now accepted and ready to land.Apr 9 2019, 1:54 PM

Closed by commit rAPI1ebf72a0269c: Fix flag admin url resolving (authored by Info-Screen). · Explain WhyApr 13 2019, 2:21 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			api/flags/admin.py (8 lines)
M			api/flags/tests.py (19 lines)

Diff 844

api/flags/admin.py

Loading...

api/flags/tests.py

Loading...