Page MenuHomePhabricator

Rename most user-facing "Autoclose" strings to "Permanent Refs"
ClosedPublic

Authored by epriestley on Apr 14 2019, 8:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 20, 2:32 PM
Unknown Object (File)
Fri, Mar 15, 3:31 PM
Unknown Object (File)
Fri, Mar 1, 5:06 AM
Unknown Object (File)
Feb 21 2024, 1:18 PM
Unknown Object (File)
Feb 4 2024, 5:05 AM
Unknown Object (File)
Feb 3 2024, 10:09 PM
Unknown Object (File)
Feb 3 2024, 10:09 PM
Unknown Object (File)
Feb 3 2024, 10:08 PM
Subscribers
None

Details

Summary

Depends on D20422. Ref T13277. Currently, "track only", "publish", and "autoclose" are three separate ideas. I'd like to generally merge them into a more natural idea called "permanent refs".

Since "Autoclose" effectively now controls both "autoclose" and "publish", rename it.

This doesn't rename all the methods or internals, and the documentation needs an update, but it renames most of the UI-facing stuff.

(You also can only specify branches as "Permanent Refs" today, but we may let you specify tags and other arbitrary refs in the future.)

Test Plan

Grepped, poked around the UI, saw UI show "Permanent" / "Permanent Refs" more often and "Autoclose" less.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley added inline comments.
src/applications/diffusion/editor/DiffusionRepositoryEditEngine.php
397–400

This kinda implies that something nefarious might happen to refs that you mark as "non permanent": are they removed from disk? Sent to the shadow realm? Maybe "only these refs are considered as permanent"?

src/applications/diffusion/management/DiffusionRepositoryBranchesManagementPanel.php
112

These variables are named inconsistently now (and variables elsewhere), which I'm guessing will get cleaned up later, but I'm also not too worried about anyway.

204

Should be "is unavailable"?

This revision is now accepted and ready to land.Apr 17 2019, 7:43 PM
  • Fix "in/is unavailable" typo.
  • Clarify that we aren't going to banish other refs to the shadow realm.
This revision was automatically updated to reflect the committed changes.