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
Unknown Object (File)
Sun, Feb 9, 3:05 AM
Unknown Object (File)
Sun, Feb 9, 3:05 AM
Unknown Object (File)
Sun, Feb 9, 3:05 AM
Unknown Object (File)
Sat, Feb 8, 4:22 PM
Unknown Object (File)
Thu, Jan 30, 6:45 AM
Unknown Object (File)
Sat, Jan 25, 1:48 PM
Unknown Object (File)
Fri, Jan 24, 6:39 PM
Unknown Object (File)
Fri, Jan 24, 6:38 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