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
F13185176: D20425.diff
Sat, May 11, 2:39 AM
Unknown Object (File)
Tue, May 7, 5:06 AM
Unknown Object (File)
Fri, May 3, 2:26 AM
Unknown Object (File)
Thu, May 2, 5:34 AM
Unknown Object (File)
Mon, Apr 29, 1:56 PM
Unknown Object (File)
Sun, Apr 28, 6:13 PM
Unknown Object (File)
Sat, Apr 27, 4:03 PM
Unknown Object (File)
Wed, Apr 24, 10:02 PM
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