Differential D12107

[native][web] Consistently use "QR" instead of "Qr"
ClosedPublic
Actions

Authored by ashoat on May 20 2024, 3:52 AM.

Details

Reviewers

bartek

Commits

rCOMM5b33a676edca: [native][web] Consistently use "QR" instead of "Qr"

Summary

Better matches our conventions around capitalizing initialisms, and makes it consistent throughout the codebase.

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.May 20 2024, 3:52 AM

Herald added a subscriber: tomek. · View Herald TranscriptMay 20 2024, 3:52 AM

ashoat added a child revision: D12108: [native][web] Get rid of unnecessary React state for QR code URL.May 20 2024, 3:54 AM

Harbormaster completed remote builds in B29044: Diff 40381.May 20 2024, 4:08 AM

ashoat requested review of this revision.May 20 2024, 4:08 AM

bartek accepted this revision.May 20 2024, 5:26 AM

This revision is now accepted and ready to land.May 20 2024, 5:26 AM

Closed by commit rCOMM5b33a676edca: [native][web] Consistently use "QR" instead of "Qr" (authored by ashoat). · Explain WhyMay 21 2024, 1:11 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM5b33a676edca: [native][web] Consistently use "QR" instead of "Qr".

Revision Contents
Changeset List

Path

Size

native/

qr-code/

qr-code-screen.react.js

4 lines

web/

account/

qr-code-login.react.js

8 lines

app.react.js

4 lines

Diff 40381

View Options

native/qr-code/qr-code-screen.react.js

View Options

web/account/qr-code-login.react.js

View Options

[native][web] Consistently use "QR" instead of "Qr"ClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 40381

native/qr-code/qr-code-screen.react.js

web/account/qr-code-login.react.js

web/app.react.js

[native][web] Consistently use "QR" instead of "Qr"
ClosedPublic
Actions

Revision Contents
Changeset List