Skip to content

feat: update cr setup #598

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

Open
wants to merge 10 commits into
base: dev
Choose a base branch
from
Open

feat: update cr setup #598

wants to merge 10 commits into from

Conversation

MoKamall
Copy link
Collaborator

@MoKamall MoKamall commented Jul 7, 2025

Description of the change

Added InstabugWidget, a wrapper used to wrap the main application to provide out of the box automatic Crash Reporting support.

Example:

void main() {
  WidgetsFlutterBinding.ensureInitialized();

      Instabug.init(
        token: 'APP_TOKEN',
        invocationEvents: [InvocationEvent.floatingButton],
        debugLogsLevel: LogLevel.verbose,
      );

  final app = InstabugWidget(
    child: const MyApp(),
  );

  runApp(app);
}

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

https://instabug.atlassian.net/browse/MOB-19013

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

@InstabugCI
Copy link
Collaborator

InstabugCI commented Jul 7, 2025

Coverage Report

Label Coverage Status
Dart 83.8%

Generated by 🚫 dangerJS against 5210877

@MoKamall MoKamall changed the title Feat/update cr setup feat: update cr setup Jul 7, 2025
@AndrewAminInstabug
Copy link
Contributor

@MoKamall This PR seems to include more changes than intended. It should focus solely on adding the InstabugWidget wrapper, but it also contains commits related to the network body size and crashes. Could you separate the necessary commits to make it easier to review?

MoKamall added 10 commits July 8, 2025 18:45
- Introduced `InstabugWidget` to wrap the root of the application, providing custom error handling for both Flutter and platform errors.
- Implemented error reporting to Instabug with options for user-defined error handlers.
- Ensured that any exceptions in custom handlers are logged without disrupting Instabug's reporting functionality.
- Added `nonFatalFlutterErrors` and `shouldExitOnFlutterError` properties to `InstabugWidget` for improved error management.
- Updated error reporting logic to handle non-fatal crashes and conditionally exit the app on Flutter errors.
- Added `nonFatalExceptionLevel` property to `InstabugWidget` for better granularity in error reporting.
@MoKamall
Copy link
Collaborator Author

MoKamall commented Jul 8, 2025

@AndrewAminInstabug Please check again now.

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