Page MenuHomePhabricator
Differential D20425

Move the repository "Publishing" option to the "Basics" panel in repository management
ClosedPublic
Actions

Authored by epriestley on Apr 15 2019, 1:29 PM.
Tags
None
Referenced Files
F19508731: D20425.diff
Sat, Jan 10, 9:28 AM
F19218150: D20425.id.diff
Wed, Dec 17, 8:51 AM
F19205972: D20425.diff
Tue, Dec 16, 8:39 PM
F19046623: D20425.id48780.diff
Nov 27 2025, 11:11 AM
F18940168: D20425.diff
Nov 11 2025, 8:02 AM
F18877839: D20425.id.diff
Nov 6 2025, 12:20 PM
F18875698: D20425.diff
Nov 5 2025, 6:48 PM
F18759939: D20425.id48780.diff
Oct 6 2025, 6:57 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13277: In repositories, realign "Track Only", "Autoclose", and "Publish/Notify" toward "Permanent Refs"
Commits
rP7a4ef2bad876: Move the repository "Publishing" option to the "Basics" panel in repository…
Summary

Depends on D20424. Ref T13277. Now that the "Actions" panel only has one item ("Publishing"), just move it to the "Basics" panel.

Update the UI to show active/publishing status more clearly and relate them to one another and importing state.

Test Plan

Screen Shot 2019-04-15 at 6.27.57 AM.png (316×1 px, 52 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 15 2019, 1:29 PM
Harbormaster completed remote builds in B22597: Diff 48727.Apr 15 2019, 1:31 PM
epriestley requested review of this revision.Apr 15 2019, 1:31 PM
epriestley added a child revision: D20426: Expose repository ref rules via "diffusion.repository.search".Apr 15 2019, 1:57 PM
amckinley accepted this revision.Apr 17 2019, 7:58 PM
amckinley added inline comments.
src/applications/diffusion/controller/DiffusionRepositoryEditPublishingController.php
48

Extra "or".

62

Add a "Note: existing unpublished commits will not be published"?

This revision is now accepted and ready to land.Apr 17 2019, 7:58 PM
epriestley updated this revision to Diff 48779.Apr 18 2019, 12:22 PM
  • Wordsmithing.
  • Clarify that "reactivate" is not retroactive.
Harbormaster completed remote builds in B22651: Diff 48779.Apr 18 2019, 12:23 PM
Closed by commit rP7a4ef2bad876: Move the repository "Publishing" option to the "Basics" panel in repository… (authored by epriestley). · Explain WhyApr 18 2019, 12:24 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 48780

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/diffusion/application/PhabricatorDiffusionApplication.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionRepositoryController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionRepositoryEditPublishingController.php

Loading...
View Options

src/applications/diffusion/management/DiffusionRepositoryBasicsManagementPanel.php

Loading...
View Options

src/applications/diffusion/management/DiffusionRepositoryPublishingManagementPanel.php

Loading...
Phabricator · Built by Phacility