Skip to content

Conversation

@mini-bomba
Copy link
Contributor

@mini-bomba mini-bomba commented Aug 21, 2025

  • I agree to license my contribution under GPL-3.0 and agree to allow distribution on app stores as outlined in LICENSE-APPSTORE

To test this pull request, follow the instructions in the wiki.


  • pass errors from background threads back to clients
  • log all HTTP request errors and display them to the user where possible

Depends on: ajayyy/ExtensionTranslations#97, ajayyy/maze-utils#15

Screenshots

logging js errors on both ends

image image

logging HTTP errors

image image

popup (refresh button)

image image

popup (segment voting)

image image image

skip notice

image image image

@mini-bomba mini-bomba force-pushed the error_handling branch 2 times, most recently from 87f4785 to de8c963 Compare August 21, 2025 18:06
@mini-bomba mini-bomba marked this pull request as ready for review August 21, 2025 18:10
@mini-bomba
Copy link
Contributor Author

this is also once again related to #2272 as "error code -1" will no longer exist, instead being replaced by a hopefully more descriptive message of the error

- pass errors from background threads back to clients
- log all HTTP request errors and display them to the user where
  possible
@ajayyy ajayyy merged commit 9a526c5 into ajayyy:master Aug 29, 2025
2 checks passed
@mini-bomba mini-bomba deleted the error_handling branch August 29, 2025 09:27
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.

2 participants