Skip to content

Conversation

@Linhead
Copy link
Collaborator

@Linhead Linhead commented Oct 20, 2025

Description

Fixed console output for expected node errors (AllNodesOfflineError and AllNodesDisabledError). When all cryptocurrency nodes are disabled or offline, the browser console now displays a clean, readable message instead of cluttering it with full error stack traces.

Related issue

Closes #902

External links (optional)

N/A

Screenshots or videos (optional)

N/A

Breaking changes

No breaking changes. This only affects console output formatting for specific error types.

How to test

  1. Open the application in a browser with DevTools console open
  2. Go to Settings → Nodes
  3. Disable all nodes for any cryptocurrency (e.g., DASH, BTC, ETH)
  4. Navigate to chat list or a specific chat
  5. Expected result: Console shows only clean messages like dash: All nodes are disabled without stack traces

Notes for reviewers

  • This is a simple, focused change affecting only error display
  • The stack property override is a standard JavaScript pattern for customizing error output
  • Only affects AllNodesOfflineError and AllNodesDisabledError - all other errors remain unchanged
  • The actual Error object remains intact with all properties; only console display is affected

Checklist

  • Code is formatted
  • Tests added/updated (not needed)
  • Documentation updated (not needed)
  • Changes tested in all relevant environments

@Linhead Linhead requested a review from a team as a code owner October 20, 2025 08:28
@vercel
Copy link

vercel bot commented Oct 20, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
adamant-im Ready Ready Preview Comment Oct 20, 2025 8:28am

💡 Enable Vercel Agent with $100 free credit for automated AI reviews

@github-actions
Copy link

Deployed to https://msg-adamant-pr-906.surge.sh 🚀

@Linhead Linhead added the Nodes ADM node software issues, APIs, connectivity, consensus label Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Nodes ADM node software issues, APIs, connectivity, consensus

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Chore] Don’t show errors in the debug console when no coin nodes

2 participants