Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ios): report fatal crashes with the updated api #1290

Merged
merged 2 commits into from
Sep 30, 2024
Merged

Conversation

a7medev
Copy link
Contributor

@a7medev a7medev commented Sep 30, 2024

Description of the change

The API for reporting JavaScript crashes on iOS has been changed from Instabug.reportCrashWithStackTrace to CrashReporting.cp_reportFatalCrashWithStackTrace in the native iOS SDK as part of v13.1.0 but the React Native SDK wasn't updated to use the new API. This led to fatal JavaScript crashes being reported by the iOS SDK as native crashes rather than JavaScript crashes breaking symbolication for these crashes.

This PR updates the API to be the new one to fix this bug.

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

Jira ID: MOB-16163

Checklists

Development

  • Lint rules pass locally
  • The code changed/added as part of this pull request has been covered with tests

Code review

  • This pull request has a descriptive title and information useful to a reviewer
  • Issue from task tracker has a link to this pull request

@a7medev a7medev self-assigned this Sep 30, 2024
@a7medev a7medev changed the title fix(ios): report fatal crashes with new api fix(ios): report fatal crashes with the updated api Sep 30, 2024
@InstabugCI
Copy link
Collaborator

Coverage Report

Label Coverage Status
JavaScript 98.4%
Android 50.2%
iOS 49.2%

Generated by 🚫 dangerJS against 1f3f79a

@a7medev a7medev merged commit 3508378 into dev Sep 30, 2024
10 of 12 checks passed
@a7medev a7medev deleted the fix/ios-fatal-crashes branch September 30, 2024 11:31
YoussefFouadd pushed a commit that referenced this pull request Oct 16, 2024
* fix(ios): report fatal crashes with new api

* chore: update changelog

Signed-off-by: YoussefFouadd <[email protected]>
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.

4 participants