Skip to content

Archie/unhandled exceptions#1715

Open
archiepeterjarvis wants to merge 26 commits intomasterfrom
archie/unhandled_exceptions
Open

Archie/unhandled exceptions#1715
archiepeterjarvis wants to merge 26 commits intomasterfrom
archie/unhandled_exceptions

Conversation

@archiepeterjarvis
Copy link
Copy Markdown
Contributor

@archiepeterjarvis archiepeterjarvis commented Apr 29, 2024

Send logs over pika on unhandled exception

  • Ready for review

Checklist

  • I have completed a self review
  • I have set the recent milestone
  • I have updated the jira ticket
  • I have added the relevant labels to this PR
  • I have updated documentation (if applicable)
  • I have run the unit tests suite and they pass
  • I have updated the change log

Description

When the software runs into an unhandled exception, the log file is copied to 'crash.log' so that the software can detect that the machine crashed on the next boot. When it restarts, the user will be prompted as to whether they want to send the log file. If they choose to, it will be automatically sent over pika to the smart manager server and will be available to us & customer support.

Notes

Does the serial number get output to the logs? If so, we need to remove it for GDPR, unless we're stating otherwise?

The text on the popup needs to be bulked out a bit.
image

Dependencies for merge

Testing

Visual Test

  • Not applicable
  • Own computer
  • Console 7"
  • Console 10" (needs scaling still)

Function Test

  • Not applicable
  • Own computer
  • Console 7"
  • Console 10"

Unit Tests

  • Not applicable
  • Completed

Screenshots (if applicable)

@archiepeterjarvis archiepeterjarvis marked this pull request as draft April 29, 2024 15:41
@archiepeterjarvis archiepeterjarvis marked this pull request as ready for review May 7, 2024 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants