Page MenuHomePhabricator

Rename "beta" to "prototype" and document support policy
ClosedPublic

Authored by epriestley on Sep 14 2014, 4:37 PM.
Tags
None
Referenced Files
F12805863: D10493.id25218.diff
Wed, Mar 27, 5:38 PM
F12805685: D10493.id25218.diff
Wed, Mar 27, 5:27 PM
Unknown Object (File)
Sun, Mar 24, 12:31 AM
Unknown Object (File)
Feb 16 2024, 10:58 AM
Unknown Object (File)
Feb 15 2024, 12:02 PM
Unknown Object (File)
Feb 10 2024, 4:58 AM
Unknown Object (File)
Feb 7 2024, 12:51 PM
Unknown Object (File)
Feb 3 2024, 12:45 AM

Details

Summary

Fixes T6084. Changes:

  • Rename phabricator.show-beta-applications to phabricator.show-prototypes, to reinforce that these include early-development applications.
  • Migrate the config setting.
  • Add an explicit "no support" banner to the config page.
  • Rename "Beta" to "Prototype" in the UI.
  • Use "bomb" icon instead of "half star" icon.
  • Document prototype applications in more detail.
  • Explicitly document that we do not support these applications.
Test Plan
  • Ran migration.
  • Resolved "obsolete config" issue.
  • Viewed config setting.
  • Browsed prototypes in Applications app.
  • Viewed documentation.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Rename "beta" to "prototype" and document support policy.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: btrahan, chad.
  • On application detail screen, link "Prototype" property to prototype docs.

Looks good to me.

src/applications/config/check/PhabricatorSetupCheckExtraConfig.php
196

"existing"

src/applications/config/option/PhabricatorCoreConfigOptions.php
114

This renders as:

To learn more about prototypes, see <a href="https://secure.phabricator.com/diviner/find/?name=User+Guide%3A+Prototype+Applications&amp;type=article&amp;jump=1" target="_blank" rel="noreferrer">User Guide: Prototype Applications</a>.
  • Add "general interest" to feedback, since this is also useful.

This renders as:

Obviously intended, to reinforce the prototype nature of these applications!

hach-que added a reviewer: hach-que.

Other than the link rendering, this works as intended (from a technical standpoint anyway).

This revision is now accepted and ready to land.Sep 14 2014, 4:53 PM
epriestley removed a reviewer: hach-que.
  • Fix link in config help text.
  • Label the options "Enable/Disable Prototypes" instead of "Install/Uninstall Prototypes", to make the distinction between this and the "Install" action more clear.
This revision now requires review to proceed.Sep 14 2014, 4:53 PM

That removal was a race condition but I'm not going to file it until I hit it again or some other user hits it, that's the first time I've seen it, it's pretty minor, and it's not really avoidable in the general case with --edit.

btrahan edited edge metadata.
btrahan added inline comments.
src/applications/config/option/PhabricatorCoreConfigOptions.php
109

because they are often not

src/docs/user/userguide/prototypes.diviner
45

Maybe add something here or a third bullet like "NOTE: We may not respond to you until we reach the point in prototype development where the feedback becomes relevant."

This revision is now accepted and ready to land.Sep 16 2014, 6:30 PM
This revision was automatically updated to reflect the committed changes.