Page MenuHomePhabricator

Macro - allow view controller to be viewed publicly
ClosedPublic

Authored by btrahan on Aug 6 2014, 10:10 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 16, 3:47 AM
Unknown Object (File)
Sat, Dec 7, 9:33 PM
Unknown Object (File)
Fri, Dec 6, 6:15 AM
Unknown Object (File)
Nov 19 2024, 4:13 PM
Unknown Object (File)
Nov 8 2024, 1:14 PM
Unknown Object (File)
Nov 8 2024, 12:21 PM
Unknown Object (File)
Oct 29 2024, 5:14 PM
Unknown Object (File)
Oct 25 2024, 8:53 PM
Subscribers

Details

Summary

Fixes T5735, setting up Phacility for huge financial success.

Test Plan

opened up Safari - who logs in with Safari anyway? - and could still view a macro
could also view the list of macros

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan retitled this revision from to Macro - allow view controller to be viewed publically.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
btrahan retitled this revision from Macro - allow view controller to be viewed publically to Macro - allow view controller to be viewed publicly.Aug 6 2014, 10:11 PM
epriestley edited edge metadata.

HEY I USE SAFARI ALL THE TIME

This revision is now accepted and ready to land.Aug 6 2014, 10:16 PM

hahaha, well, at least we know you vigorously read test plans. =D

btrahan updated this revision to Diff 24465.

Closed by commit rP1bd714f8ac48 (authored by @btrahan).