Skip to content

Add ReactSoftException in ReactHostImpl only when onActivityResult, onNewIntent and onWindowFocusChange do not have the context - #44155

Closed
arushikesarwani94 wants to merge 1 commit into
react:mainfrom
arushikesarwani94:export-D56325407
Closed

Add ReactSoftException in ReactHostImpl only when onActivityResult, onNewIntent and onWindowFocusChange do not have the context#44155
arushikesarwani94 wants to merge 1 commit into
react:mainfrom
arushikesarwani94:export-D56325407

Conversation

@arushikesarwani94

Copy link
Copy Markdown
Contributor

Summary:
Add ReactSoftException in ReactHostImpl only when onActivityResult, onNewIntentand onWindowFocusChange do not have the context

Changelog:
[Android][Fixed] ReactSoftExceptions in ReactHostImpl only when Context is null

Differential Revision: D56325407

@facebook-github-botfacebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 18, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D56325407

@analysis-bot

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a19,410,453-110
androidhermesarmeabi-v7an/a--
androidhermesx86n/a--
androidhermesx86_64n/a--
androidjscarm64-v8a22,784,129-70
androidjscarmeabi-v7an/a--
androidjscx86n/a--
androidjscx86_64n/a--

Base commit: 4fbc1f2
Branch: main

…nNewIntent and onWindowFocusChange do not have the context (react#44155)
Summary:
Add ReactSoftException in ReactHostImpl only when `onActivityResult`, `onNewIntent`and `onWindowFocusChange` do not have the context
Changelog:
[Android][Fixed] ReactSoftExceptions in ReactHostImpl only when Context is null
Differential Revision: D56325407
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D56325407

arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Apr 19, 2024
…yResult, onNewIntent and onWindowFocusChange do not have the context (react#44155)
Summary:
Add ReactSoftException in ReactHostImpl only when `onActivityResult`, `onNewIntent`and `onWindowFocusChange` do not have the context
Changelog:
[Android][Fixed] ReactSoftExceptions in ReactHostImpl only when Context is null
Reviewed By: cortinico
Differential Revision: D56325407
@facebook-github-botfacebook-github-bot added the Merged This PR has been merged. label Apr 19, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request has been merged in ac3261f.

@github-actions

Copy link
Copy Markdown

This pull request was successfully merged by @arushikesarwani94 in ac3261f.

When will my fix make it into a release? | How to file a pick request?

cipolleschi pushed a commit that referenced this pull request Jun 3, 2024
…nNewIntent and onWindowFocusChange do not have the context (#44155)
Summary:
Pull Request resolved: #44155
Add ReactSoftException in ReactHostImpl only when `onActivityResult`, `onNewIntent`and `onWindowFocusChange` do not have the context
Changelog:
[Android][Fixed] ReactSoftExceptions in ReactHostImpl only when Context is null
Reviewed By: cortinico
Differential Revision: D56325407
fbshipit-source-id: a9f8fd5772fc05d39e72236fb8edfe5f8a9d6a43
This was referenced Jun 28, 2024
@cipolleschicipolleschi mentioned this pull request Jul 22, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.fb-exportedMergedThis PR has been merged.p: FacebookPartner: FacebookPartner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@arushikesarwani94@facebook-github-bot@analysis-bot