Skip to content

Commit ea7ce89

Browse files
author
Dave MacFarlane
committed
fix compile and explicitly name acronym in link
1 parent 00a54cb commit ea7ce89

File tree

3 files changed

+15
-13
lines changed

3 files changed

+15
-13
lines changed

jsx/MFAPrompt.tsx

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,17 @@ function Digit(props: {
4343
}
4444

4545
type errorCallback = (msg: string) => void;
46+
type MFACode = [
47+
number|null,
48+
number|null,
49+
number|null,
50+
number|null,
51+
number|null,
52+
number|null];
53+
4654
/**
47-
* Prompt for a multi-factor authentication code and call onValidate
48-
* after a valid code has been entered.
55+
* Prompt for a multi-factor authentication code and call validate
56+
* callback to validate the code after all 6 digits have been entered.
4957
*
5058
* @param props - React props
5159
* @param props.validate - Callback when a code is entered to validate it.
@@ -55,18 +63,12 @@ type errorCallback = (msg: string) => void;
5563
function MFAPrompt(props: {validate:
5664
(code: string, onError: errorCallback) => void
5765
}) {
58-
const [code, setCode] = useState<[
59-
number|null,
60-
number|null,
61-
number|null,
62-
number|null,
63-
number|null,
64-
number|null]>([null, null, null, null, null, null]);
66+
const [code, setCode] = useState<MFACode>([null, null, null, null, null, null]);
6567
const digitCallback = useCallback(
6668
(index: number, value: number): boolean => {
6769
if (value >= 0 && value <= 9) {
6870
setCode(prev => {
69-
const newCode = [...prev];
71+
const newCode: MFACode = [...prev];
7072
newCode[index] = value;
7173
return newCode;
7274
});

modules/my_preferences/jsx/mfa.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,8 @@ function MFAIndex(): React.ReactElement {
9090
setup MFA in LORIS!</b>
9191
</p>
9292
<QRCode value={mfaUrl} />
93-
<p>Can't scan the QR code?
94-
<a href="#" onClick={() => setShowModal(true)}>Setup manually.</a>
93+
<p>Can't scan the QR code? <a href="#" onClick={() => setShowModal(true)}>
94+
Setup manually.</a>
9595
</p>
9696
<CodeValidator secret={key} />
9797
</div>;

modules/my_preferences/templates/form_my_preferences.tpl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@
6969
in a different form element with a smarty template and it's easier
7070
to create a new "fresh" page with modern react/etc than rewrite
7171
the whole page or do a hybrid here *}
72-
<a href="{$baseurl}/my_preferences/mfa">Configure MFA</a>
72+
<a href="{$baseurl}/my_preferences/mfa">Configure multi-factor authentication (MFA)</a>
7373
</label>
7474
</div>
7575

0 commit comments

Comments
 (0)