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

Diagnostics 2.0 #103

Merged
merged 4 commits into from
Nov 1, 2021
Merged

Diagnostics 2.0 #103

merged 4 commits into from
Nov 1, 2021

Conversation

AvdLee
Copy link
Contributor

@AvdLee AvdLee commented Oct 29, 2021

This is our first major improvement of the Diagnostics report we've been using for more than a year now at WeTransfer. Based on our experience, these changes are needed to make it easier to find out what's wrong with the user that reported in.

The main goal of these changes is to speed up the time from a bug report to a fix. 🚀

Changes in action:

Screen.Recording.2021-10-29.at.15.56.32.mov

- Added filter options to hide system, errors, and debug logs
- Added buttons to collapse/expand all sessions
- Proper support for logging HTML links
- Fixed CSS for links
@AvdLee AvdLee marked this pull request as ready for review October 29, 2021 14:20
@AvdLee AvdLee requested a review from kevinrenskers October 29, 2021 14:20
@wetransferplatform
Copy link
Collaborator

wetransferplatform commented Oct 29, 2021

Messages
📖

View more details on Bitrise

📖 DiagnosticsTests: Executed 18 tests, with 0 failures in 0.340 seconds

Code Coverage Report

Name Coverage
Diagnostics 71.39% ⚠️

Generated by 🚫 Danger Swift against 10e2ce8

Copy link
Contributor

@amirdew amirdew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, checking a report is much easier now

@AvdLee AvdLee merged commit 116227a into master Nov 1, 2021
@AvdLee AvdLee deleted the feature/diagnostics-2-0 branch November 1, 2021 08:27
@wetransferplatform
Copy link
Collaborator

Congratulations! 🎉 This was released as part of Release 2.0.0 🚀

Generated by GitBuddy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants