[native][web] Fix crash due to getConfig being called before ready
ClosedPublic
Actions

Authored by ashoat on Aug 22 2024, 1:03 PM.

Details

Reviewers

varun
will

Commits

rCOMM2ac10ed878f2: [native][web] Fix crash due to getConfig being called before ready

Summary

This addresses ENG-9092; additional context there.

Test Plan

Confirmed the problem went away

Diff Detail

Repository

rCOMM Comm

Branch

ashoat/fix

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Aug 22 2024, 1:03 PM

Herald added a subscriber: tomek. · View Herald TranscriptAug 22 2024, 1:03 PM

ashoat published this revision for review.Aug 22 2024, 1:04 PM

varun accepted this revision.Aug 22 2024, 1:05 PM

This revision is now accepted and ready to land.Aug 22 2024, 1:05 PM

Harbormaster completed remote builds in B31223: Diff 43594.Aug 22 2024, 1:21 PM

Closed by commit rCOMM2ac10ed878f2: [native][web] Fix crash due to getConfig being called before ready (authored by ashoat). · Explain WhyAug 22 2024, 1:50 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM2ac10ed878f2: [native][web] Fix crash due to getConfig being called before ready.

Revision Contents
Changeset List

Path

Size

native/

qr-code/

qr-code-screen.react.js

5 lines

web/

account/

qr-code-login.react.js

5 lines

Diff 43594

View Options

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

This file was deleted.

This file was completely deleted. Show File Contents