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

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

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Sep 25, 1:30 AM

	Unknown Object (File)
	Wed, Sep 25, 1:30 AM

	Unknown Object (File)
	Mon, Sep 23, 12:38 AM

	Unknown Object (File)
	Sep 11 2024, 8:26 PM

	Unknown Object (File)
	Sep 7 2024, 5:03 PM

	Unknown Object (File)
	Sep 7 2024, 2:02 AM

	Unknown Object (File)
	Sep 6 2024, 11:31 AM

	Unknown Object (File)
	Sep 4 2024, 1:00 PM

View All 25 Files

Subscribers

tomek

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