Page MenuHomePhabricator

[native] Fix LoggedOutModal buttons not appearing on load in release build
ClosedPublic

Authored by ashoat on Wed, May 29, 7:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jun 22, 1:28 AM
Unknown Object (File)
Tue, Jun 18, 2:02 AM
Unknown Object (File)
Fri, Jun 14, 8:13 AM
Unknown Object (File)
Wed, Jun 12, 7:46 PM
Unknown Object (File)
Wed, Jun 12, 12:44 AM
Unknown Object (File)
Mon, Jun 10, 8:56 AM
Unknown Object (File)
Mon, Jun 10, 4:35 AM
Unknown Object (File)
Sun, Jun 9, 11:36 PM
Subscribers

Details

Summary

This resolves ENG-8321.

I made a mistake in D12208. In Reanimated 3 it's okay to pass a Reanimated value directly to a style prop of an Animated.View, but not in Reanimated 2. I had been reading the Reanimated 3 docs initially, and after learning about this I updated most places but seem to have missed this spot.

In Reanimated 2, all uses of Reanimated values must be wrapped in useAnimatedStyle.

Guessing it only shows up in a release build due to some race conditions.

Test Plan

Confirmed that it resolves the linked Linear issue by compiling a release build to my iOS simulator

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable