Skip to content

fix: brownfield integration#6774

Merged
WoLewicki merged 3 commits intomainfrom
@wolewicki/handle-browfield
Dec 4, 2024
Merged

fix: brownfield integration#6774
WoLewicki merged 3 commits intomainfrom
@wolewicki/handle-browfield

Conversation

@WoLewicki
Copy link
Member

@WoLewicki WoLewicki commented Nov 29, 2024

Summary

In brownfield, // we need only one instance because SurfacePresenter is the same during the application lifetime is false since you can destroy RN and create it again.

Test plan

Run Reanimated in brownfield and try to create and destroy the whole RN a couple of times.

@piaskowyk piaskowyk self-requested a review November 29, 2024 17:38
Copy link
Member

@piaskowyk piaskowyk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's merge this as a fix, especially since we've removed support for Bridgeful mode on Fabric 👍

@WoLewicki WoLewicki added this pull request to the merge queue Dec 4, 2024
Merged via the queue into main with commit cf1f618 Dec 4, 2024
@WoLewicki WoLewicki deleted the @wolewicki/handle-browfield branch December 4, 2024 16:51
tomekzaw pushed a commit that referenced this pull request Dec 9, 2024
## Summary

In brownfield, ` // we need only one instance because SurfacePresenter
is the same during the application lifetime` is false since you can
destroy RN and create it again.

## Test plan
Run Reanimated in brownfield and try to create and destroy the whole RN
a couple of times.
@tomekzaw tomekzaw mentioned this pull request Dec 9, 2024
tomekzaw pushed a commit that referenced this pull request Dec 9, 2024
## Summary

In brownfield, ` // we need only one instance because SurfacePresenter
is the same during the application lifetime` is false since you can
destroy RN and create it again.

## Test plan
Run Reanimated in brownfield and try to create and destroy the whole RN
a couple of times.
tjzel pushed a commit that referenced this pull request Dec 13, 2024
## Summary

In brownfield, ` // we need only one instance because SurfacePresenter
is the same during the application lifetime` is false since you can
destroy RN and create it again.

## Test plan
Run Reanimated in brownfield and try to create and destroy the whole RN
a couple of times.
tjzel pushed a commit that referenced this pull request Dec 13, 2024
## Summary

In brownfield, ` // we need only one instance because SurfacePresenter
is the same during the application lifetime` is false since you can
destroy RN and create it again.

## Test plan
Run Reanimated in brownfield and try to create and destroy the whole RN
a couple of times.
tjzel pushed a commit that referenced this pull request Dec 13, 2024
## Summary

In brownfield, ` // we need only one instance because SurfacePresenter
is the same during the application lifetime` is false since you can
destroy RN and create it again.

## Test plan
Run Reanimated in brownfield and try to create and destroy the whole RN
a couple of times.
r0h0gg6 pushed a commit to r0h0gg6/react-native-reanimated that referenced this pull request Jul 28, 2025
## Summary

In brownfield, ` // we need only one instance because SurfacePresenter
is the same during the application lifetime` is false since you can
destroy RN and create it again.

## Test plan
Run Reanimated in brownfield and try to create and destroy the whole RN
a couple of times.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants