D26028.1748080221.diff
No OneTemporary
Actions

Size

1000 B

Referenced Files

None

Subscribers

None

D26028.1748080221.diff
View Options

	diff --git a/src/applications/settings/panel/PhabricatorMultiFactorSettingsPanel.php b/src/applications/settings/panel/PhabricatorMultiFactorSettingsPanel.php
	--- a/src/applications/settings/panel/PhabricatorMultiFactorSettingsPanel.php
	+++ b/src/applications/settings/panel/PhabricatorMultiFactorSettingsPanel.php
	@@ -245,10 +245,18 @@
	$menu->addItem($item);
	}

	- return $this->newDialog()
	+ $dialog = $this->newDialog()
	->setTitle(pht('Choose Factor Type'))
	->appendChild($menu)
	->addCancelButton($cancel_uri);
	+
	+ if ($viewer->getIsEnrolledInMultiFactor()) {
	+ $dialog->appendParagraph(pht(
	+ 'Note: You already have an Auth Factor configured. Adding '.
	+ 'another factor will require you to always provide all Auth '.
	+ 'Factors instead of selecting one of your Auth Factors.'));
	+ }
	+ return $dialog;
	}

	// NOTE: Beyond providing guidance, this step is also providing a CSRF gate