Page MenuHomePhabricator
Differential D17151

Correct "Manage Password" link in Quickling in Diffusion
ClosedPublic
Actions

Authored by epriestley on Jan 8 2017, 3:58 PM.
Tags
None
Referenced Files
F21410918: D17151.diff
Wed, Jul 29, 12:54 PM
Unknown Object (File)
Sun, Jul 12, 5:03 PM
Unknown Object (File)
Tue, Jul 7, 6:28 AM
Unknown Object (File)
Tue, Jul 7, 6:28 AM
Unknown Object (File)
Fri, Jul 3, 12:26 AM
Unknown Object (File)
Thu, Jul 2, 7:03 AM
Unknown Object (File)
Jun 1 2026, 5:32 PM
Unknown Object (File)
Jun 1 2026, 5:32 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12080: Clicking "Manage Password" next to repo URI does not change page
Commits
rPd4248d231bed: Correct "Manage Password" link in Quickling in Diffusion
Summary

Fixes T12080. This was missing a "/", but stop hard-coding these URIs.

Test Plan

Clicked both links with Quickling as a logged-in and logged-out user, ended up in the right place.

Diff Detail

Repository
rP Phabricator
Branch
quri
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15134
Build 19873: Run Core Tests
Build 19872: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
view/
12 lines
settings/
controller/
2 lines
panel/
11 lines

Diff 41247

View Options

src/applications/diffusion/view/DiffusionCloneURIView.php

Loading...
View Options

src/applications/settings/controller/PhabricatorSettingsMainController.php

Loading...
View Options

src/applications/settings/panel/PhabricatorSettingsPanel.php

Loading...
Phabricator · Built by Phacility