Skip to content

fix(android): dont dispose fragment on onloaded - #8793

Merged
NathanWalker merged 2 commits into
NativeScript:release/8.1.0from
Akylas:feature/android_dont_dispose_unloaded
Aug 11, 2021
Merged

NathanWalker merged 2 commits into
NativeScript:release/8.1.0from
Akylas:feature/android_dont_dispose_unloaded

Conversation

@farfromrefug

Copy link
Copy Markdown
Collaborator

As explained here #6659
That hack has bad consequences.
Also i actually cant see any wrong behavior using the sample provided in the issue and upgrading to latest N.

@cla-bot cla-bot Bot added the cla: yes label Aug 27, 2020
@NathanaelA

This comment was marked as abuse.

@farfromrefug

Copy link
Copy Markdown
Collaborator Author

The initial crashing sample is not crashing anymore with this PR. At least I could not get it to crash

farfromrefug added a commit to Akylas/NativeScript that referenced this pull request Sep 16, 2020
@NathanWalker NathanWalker added the ready for test TSC needs to test this and confirm against live production apps and automated test suites label Oct 27, 2020
@NathanWalker NathanWalker added this to the 8.1 milestone Aug 6, 2021
@NathanWalker
NathanWalker changed the base branch from master to release/8.1.0 August 11, 2021 18:28
@NathanWalker
NathanWalker merged commit 03b7715 into NativeScript:release/8.1.0 Aug 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes ready for test TSC needs to test this and confirm against live production apps and automated test suites

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants