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: Adjust logging behavior based on the debugLogLevel. #1319

Open
wants to merge 10 commits into
base: feat/excule-dev-server-from-network-logs
Choose a base branch
from

Conversation

ahmedAlaaInstabug
Copy link
Contributor

Description of the change

Adjust logging behavior based on the debugLogLevel.

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-16570

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

mzelzoghbi and others added 7 commits October 2, 2024 18:23
* chore: upgrade dependencies

* refactor(example): upgrade to react native 0.75.4

* chore: integrate android sdk v14 snapshot

* ci: install cocoapods 1.14

* ci: upgrade xcode to 15.4

* chore: remove .xcode.env.local

* ci: install cocoapods into usr/local/bin

* ci: fix empty jacoco report issue
* Release : v14.0.0

* Release : v14.0.0

* Release : v14.0.0
* feat(android): add session sync callback (#1281)

* feat(android): add SRSyncCallback

* feat: implement and test syncCallback CP side

* feat(example): use SRSyncCallback in example app

* ci: fix tests

* fix: export session data type

* fix(example): use session data type

* fix(android):remove data modifier

* fix(android): add property modifiers

* fix(android): update test case

* fix: enhance test case

* fix: update session data type

* fix: add more session metadata to setSyncCallback

* fix: update syncCallback test

* feat: add launchType to session metadata for setSyncCallback

* fix: import type

* fix: assert evaluate sync returns correct value

* fix: import type

* fix: cleanup

* chore: update js doc

* fix: typo

* fix: follow interface naming convention

* fix: update type

* fix: refactor syncCallback

* fix: default syncing session to true

* fix: convert network logs to readable array

* chore: add discriptive comment

* chore: use readable map for session metadata

* fix: setSyncCallback should sync in case of exception

* fix: move SessionMetadata to models

* fix: update SessionMetadata type import

* fix: report bug e2e test

---------

Co-authored-by: Ahmed Elrefaey <68241710+a7medev@users.noreply.github.com>

* feat(ios): add session sync callback  (#1282)

* feat(android): add SRSyncCallback

* feat: implement and test syncCallback CP side

* feat(example): use SRSyncCallback in example app

* ci: fix tests

* fix: export session data type

* fix(example): use session data type

* fix(android):remove data modifier

* fix(android): add property modifiers

* fix(android): update test case

* fix: enhance test case

* fix(ios): update network log signature

* chore(ios): integrate dynamic sampling snapshot

* fix:update IOS network log unit test

* fix: update session metadata

* feat(ios): add setSyncCallback

* fix: pod.lock file

* fix: update session data type

* fix: add more session metadata to setSyncCallback

* fix: update syncCallback test

* feat: add launchType to session metadata for setSyncCallback

* fix: import type

* fix: enhance test case

* fix: add more session metadata to setSyncCallback

* fix: update syncCallback test

* feat: add launchType to session metadata for setSyncCallback

* fix: import type

* feat(ios): add launchType metadata to session syncCallback

* fix: add unknown type to launch types

* fix: assert evaluate sync returns correct value

* fix: import type

* fix: cleanup

* chore: update js doc

* fix: typo

* fix: follow interface naming convention

* fix: update type

* fix: refactor syncCallback

* fix: default syncing session to true

* fix: convert network logs to readable array

* chore: add discriptive comment

* chore: use readable map for session metadata

* fix: setSyncCallback should sync in case of exception

* fix: move SessionMetadata to models

* fix: update SessionMetadata type import

* fix: report bug e2e test

* chore (ios): update snapshot

* chore (ios): refactor callback

* fix: return network logs

* chore: update podfile.lock

* chore: fix formatting

* chore: revert Podfile.lock

* chore: fix ci

* fix: launchType typo

* fix: update class sessionEvaluationCompletion atomicity

* chore: enhance syncCallback formatting

* chore: update evaluateSync formatting

* fix: fix test SetSyncCallback

* fix: update getNetworkLogsArray return value

---------

Co-authored-by: Ahmed Elrefaey <68241710+a7medev@users.noreply.github.com>

* Revert "fix(ios): update network log signature"
This reverts commit 8d9036e.

* chore(ios): update snapshot

* fix: ios network logging test after reverting

* fix: convert sendEvent arg from writable to readable map

* chore(android): update snapshot

* fix(android): refactor getSessionMetadataMap to tolerate null values

* fix(ios): update fulfill exception wait time in test

* fix(android): convert session metadat map to readable map

* chore: update docs

* fix: remove hot launch type

* fix: increase timeout expectation in test case

* Revert "fix: increase timeout expectation in test case"

This reverts commit be32acd.

* feat(example): add features and buttons implementation (#1280)

Jira ID: RL-224

* fix(android): add unknown launch type

* chore: update documentation

* feat: upgrade to 14.0.0

* feat: upgrade to 14.0.0

* feat: upgrade to 14.0.0

* merge dev

* merge dev

* merge dev

* fix: test case

---------

Co-authored-by: Ahmed Elrefaey <68241710+a7medev@users.noreply.github.com>
Co-authored-by: YoussefFouadd <yfouad@instabug.com>
Co-authored-by: Ahmed alaa <aalaa@instabug.com>
Co-authored-by: Mohamed Zakaria El-Zoghbi <5540492+mzelzoghbi@users.noreply.github.com>
@InstabugCI
Copy link
Collaborator

InstabugCI commented Nov 18, 2024

Coverage Report

Label Coverage Status
JavaScript 97.7%
Android 50.8%
iOS 50.9%

Generated by 🚫 dangerJS against a813889

@InstabugCI
Copy link
Collaborator

Your snapshot has been generated! 🚀

Installation

You can install the snapshot through NPM:

npm install instabug-reactnative@13.4.0-1319209-SNAPSHOT

or Yarn:

yarn add instabug-reactnative@13.4.0-1319209-SNAPSHOT

@InstabugCI
Copy link
Collaborator

Your snapshot has been generated! 🚀

Installation

You can install the snapshot through NPM:

npm install instabug-reactnative@14.0.0-1319578-SNAPSHOT

or Yarn:

yarn add instabug-reactnative@14.0.0-1319578-SNAPSHOT

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.

5 participants