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
F17949331: D17151.id.diff
Thu, Jul 31, 11:16 PM
F17940296: D17151.diff
Thu, Jul 31, 4:13 AM
Unknown Object (File)
Jun 19 2025, 9:47 AM
Unknown Object (File)
Jun 18 2025, 7:10 AM
Unknown Object (File)
Jun 17 2025, 7:17 PM
Unknown Object (File)
Jun 16 2025, 12:59 PM
Unknown Object (File)
May 29 2025, 8:56 AM
Unknown Object (File)
May 8 2025, 3:08 AM
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