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

Add Radiography integration #1647

Closed
romtsn opened this issue Aug 13, 2021 · 2 comments
Closed

Add Radiography integration #1647

romtsn opened this issue Aug 13, 2021 · 2 comments

Comments

@romtsn
Copy link
Member

romtsn commented Aug 13, 2021

Add optional Radiography integration to automatically capture view hierarchy in case of a crash. Can be sent as a text file via an attachment.

The benefit is that it also supports Jetpack Compose.

@marandaneto
Copy link
Contributor

marandaneto commented Apr 25, 2022

This is partially blocked by getsentry/sentry-cocoa#1772 if we decide going with the 1st option.

  1. Ideally, we'd do a quick investigation for Cocoa support, so we'd know if it's possible and what the widget tree would look like, the reason is that we already know what the file format looks like for Android but not for iOS.

Suggesting a file format where Sentry would special case and display either a new tab or a pop-up screen (preview) specifically for radiography would enable us to develop specific features for radiography such as collapsing or hiding elements.

The UI team will need to be part of it as well.

  1. The other option is that if we go with the current file format for Android, and iOS does what's possible, if later we decide adding this feature on Sentry UI, either the event ingestion parse both files to a preferred data structure where Sentry UI handles it or we'd need to do a breaking change.

@markushi
Copy link
Member

markushi commented Feb 1, 2023

View Hierarchy is now collected without radiography.

@markushi markushi closed this as completed Feb 1, 2023
kanban automation moved this from To do to Done Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants