Skip to content

Conversation

@jacbn
Copy link
Contributor

@jacbn jacbn commented Oct 20, 2025

When the editor fails to load (exclusively due to a request timeout), we show a small warning pointing users to our contact form. This PR aims to improve this error to first point users to the FAQ page which suggests the usual fix of allowing network traffic to the code-editor.ada-cs.org domain. The original contact us link is also available for cases where this does not solve the issue.

Also adds Plausible logging to the timeout, to better understand the scale of this problem. In case there are problems with specific browsers, OSes, etc, we will also spot them more easily there.

@codecov
Copy link

codecov bot commented Oct 20, 2025

Codecov Report

❌ Patch coverage is 25.00000% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 41.55%. Comparing base (afec320) to head (37449e0).
⚠️ Report is 96 commits behind head on main.

Files with missing lines Patch % Lines
...components/content/IsaacInteractiveCodeSnippet.tsx 25.00% 12 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1788      +/-   ##
==========================================
- Coverage   41.57%   41.55%   -0.02%     
==========================================
  Files         535      535              
  Lines       23521    23533      +12     
  Branches     6940     7786     +846     
==========================================
+ Hits         9778     9779       +1     
- Misses      13702    13713      +11     
  Partials       41       41              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@axlewin axlewin merged commit 17edd32 into main Oct 23, 2025
10 checks passed
@axlewin axlewin deleted the feature/ada-editor-better-logging branch October 23, 2025 14:17
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