Page MenuHomePhabricator

Allow XHPAST to be viewed public
ClosedPublic

Authored by chad on Jul 22 2015, 5:01 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 12 2024, 6:08 PM
Unknown Object (File)
Feb 12 2024, 4:54 PM
Unknown Object (File)
Feb 12 2024, 4:54 PM
Unknown Object (File)
Feb 12 2024, 1:07 PM
Unknown Object (File)
Feb 3 2024, 5:56 AM
Unknown Object (File)
Feb 2 2024, 3:39 AM
Unknown Object (File)
Jan 30 2024, 11:09 PM
Unknown Object (File)
Jan 28 2024, 8:06 AM
Subscribers

Details

Summary

Ref T4830. Not sure this should be public (consume resources?)

Test Plan

View logged out.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Allow XHPAST to be viewed public.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jul 22 2015, 2:04 PM
epriestley edited edge metadata.

Actually, I wouldn't let them use PhabricatorXHPASTViewRunController.php.

This revision now requires changes to proceed.Jul 22 2015, 2:05 PM

Basically, this app is super oldschool. Ideally we should:

  • Add Query + SearchEngine.
  • Have a standard list, make the list public, etc.
  • Proper "Create AST" button.

We'd keep "create AST" non-public, like other creation operations in other apps.

Just doing this change (without all that extra work, and without making RunController public) is fine, but it just means that sharing links to trees will work. The landing page for the app will still be nonfunctional for public users.

chad edited edge metadata.
  • remove from run controller
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jul 22 2015, 8:22 PM
This revision was automatically updated to reflect the committed changes.