Skip to content

feat: update sample app #1400

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 7 commits into
base: dev
Choose a base branch
from
Open

feat: update sample app #1400

wants to merge 7 commits into from

Conversation

MoKamall
Copy link

@MoKamall MoKamall commented Jun 1, 2025

Description of the change

Update the BR, and CR pages in the Sample App to provide better support for Appium SDK Automation

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

Checklists

  • Update BR Screen
  • Update CR Screen

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

@MoKamall MoKamall changed the base branch from master to dev June 1, 2025 10:46
@InstabugCI
Copy link
Collaborator

InstabugCI commented Jun 2, 2025

Warnings
⚠️ Coverage for JavaScript is less than the expected threshold of 90%

Coverage Report

Label Coverage Status
JavaScript 88.5%
Android 50.3%
iOS 46.6%

Generated by 🚫 dangerJS against 74c0f99

ahmedAlaaInstabug and others added 7 commits June 2, 2025 22:36
Set extended bug reporting options

Disable/Enable bug reporting

Set the disclaimer text

Set the post sending dialog

Set InvocationOption API
- Added `BugReportingStateScreen` to toggle between enabled and disabled states for bug reporting.
- Updated `ListTile` component to support subtitles for better UI clarity.
- Adjusted navigation to include new bug reporting screens.
- Added new screens for managing bug reporting types, disclaimer text, invocation events, options, and session profiler.
- Updated `ListTile` component to support subtitle truncation.
- Integrated new bug reporting state management features into navigation.
- Refactored `BugReportingScreen` to streamline state handling and improve UI interactions.
- Introduced `ViewHierarchyScreen` and `RepliesStateScreen` for managing respective states.
- Updated `HomeStack` to include new screens in navigation.
- Enhanced `BugReportingScreen` with options to navigate to the new screens and manage their states.
- Added state management for enabling/disabling View Hierarchy and Replies features.
- Modified `sendNativeNonFatal` method to remove the parameter and use a test exception directly.
- Introduced `CrashReportingStateScreen`, `NDKCrashesStateScreen`, `NonFatalCrashesScreen`, and `FatalCrashesScreen` for managing crash reporting states and handling different crash types.
- Updated `HomeStack` to include new screens for crash reporting.
- Enhanced `CrashReportingScreen` to navigate to the new state management screens and improved UI interactions.
- Added test IDs for better integration with testing frameworks.
- Introduced `UserConsentScreen` for managing user consent details, including key, description, mandatory status, checked status, and action type.
- Updated `HomeStack` to include the new `UserConsent` route.
- Enhanced `BugReportingScreen` with a new option to navigate to the `UserConsentScreen`.
- Added test IDs for improved testing integration.
@MoKamall MoKamall force-pushed the feat/update-sample-app branch from cf82748 to 74c0f99 Compare June 2, 2025 19:38
@MoKamall MoKamall self-assigned this Jun 7, 2025
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.

3 participants