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

Integrate advanced Bugsee features #49

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

koukibadr
Copy link
Contributor

GitHub Issue: #

Proposed Changes

  • Bug fix
  • Feature
  • Code style update (formatting)
  • Refactoring (no functional changes, no api changes)
  • Build or CI related changes
  • Documentation content changes
  • Other, please describe:

Description

  • Implement top-level Bugsee exception handler for dart and flutter (layout) exceptions
  • Implement data obscuration in reported videos
  • Add data obscuration flag saved in shared prefs (updated from the diagnostic overlay)
  • Custom logs with filters and enable/disable flag in shared prefs.
  • Enable custom events logged with reported exception
  • Attach custom traces with reported exceptions
  • Attach log file with the reported exceptions
  • Update diagnostic layout to test all Bugsee features (data obscuration, filter, events, traces, attributes...)

Impact on version

  • Major
    • The template structure was changed.
  • Minor
    • New functionalities were added.
  • Patch
    • A bug in behavior was fixed.
    • Documentation was changed.

PR Checklist

Always applicable

No matter your changes, these checks always apply.

  • Your conventional commits are aligned with the Impact on version section.
  • Updated CHANGELOG.md.
    • Use the latest Major.Minor.X header if you do a Patch change.
    • Create a new Major.Minor.X header if you do a Minor or Major change.
    • If you create a new header, it aligns with the Impact on version section and matches what is generated in the build pipeline.
  • Documentation files were updated according with the changes.
    • Update README.md and src/cli/CLI.md if you made changes to templating.
    • Update AzurePipelines.md and src/app/README.md if you made changes to pipelines.
    • Update Diagnostics.md if you made changes to diagnostic tools.
    • Update Architecture.md and its diagrams if you made architecture decisions or if you introduced new recipes.
    • ...and so forth: Make sure you update the documentation files associated to the recipes you changed. Review the topics by looking at the content of the doc/ folder.
  • Images about the template are referenced from the wiki and added as images in this git.
  • TODO comments are hints for next steps for users of the template and not planned work.

Contextual

Based on your changes these checks may not apply.

  • Automated tests for the changes have been added/updated.
  • Tested on all relevant platforms

Other information

Internal Issue (If applicable):

@koukibadr koukibadr requested review from jeanplevesque and Soap-141 and removed request for jeanplevesque November 21, 2024 15:16
@koukibadr koukibadr closed this Nov 22, 2024
@koukibadr koukibadr reopened this Nov 22, 2024
@koukibadr koukibadr marked this pull request as draft November 22, 2024 13:34
@jeanplevesque
Copy link
Member

Make sure you also have a Bugsee.md file under the doc folder to explain how Bugsee is integrated.

@koukibadr koukibadr marked this pull request as ready for review November 28, 2024 11:36
@koukibadr koukibadr force-pushed the config/bk/integrate-advanced-bugsee branch from 139444c to 934f7f6 Compare December 2, 2024 11:00
@jeanplevesque jeanplevesque requested a review from a team December 2, 2024 20:42
@carlh98
Copy link
Contributor

carlh98 commented Feb 25, 2025

@koukibadr Merge latest main in your branch

@koukibadr koukibadr requested a review from carlh98 February 27, 2025 16:32
@koukibadr koukibadr requested a review from carlh98 March 3, 2025 13:41
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