kolibri icon indicating copy to clipboard operation
kolibri copied to clipboard

Distributed error reporting: Capture more information about the error and its environment

Open thesujai opened this issue 1 year ago • 1 comments

Summary

  1. Changes the model ErrorReports by renaming some fields like error_from to category and more.
  2. Adds new JSON fields for backend and frontend so we can store more contexts of error into them
  3. Update the remaining part of the code with the model changes, by update in backend error handling and frontend error handling mechanism

References

Closes #12376

Reviewer guidance

Cannot find a way yo get the node version in frontend


Testing checklist

  • [x] Contributor has fully tested the PR manually
  • [ ] If there are any front-end changes, before/after screenshots are included
  • [ ] Critical user journeys are covered by Gherkin stories
  • [x] Critical and brittle code paths are covered by unit tests

PR process

  • [x] PR has the correct target branch and milestone
  • [ ] PR has 'needs review' or 'work-in-progress' label
  • [ ] If PR is ready for review, a reviewer has been added. (Don't use 'Assignees')
  • [ ] If this is an important user-facing change, PR or related issue has a 'changelog' label
  • [ ] If this includes an internal dependency change, a link to the diff is provided

Reviewer checklist

  • Automated test coverage is satisfactory
  • PR is fully functional
  • PR has been tested for accessibility regressions
  • External dependency files were updated if necessary (yarn and pip)
  • Documentation is updated
  • Contributor is in AUTHORS.md

thesujai avatar Jul 01 '24 08:07 thesujai

This is looking good in general - and I think most of the previous request for changes have been addressed, I just noticed one more thing around error handling.

rtibbles avatar Aug 01 '24 22:08 rtibbles

Updated the PR

thesujai avatar Aug 02 '24 05:08 thesujai