Skip to content

Add configuration option to suppress RPC connection error notifications #357

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

MrMinemeet
Copy link
Contributor

Add setting in config to suppress the RPC: RPC_COULD_NOT_CONNECT error.

Useful if Discord is not permanently running (e.g. no auto start-up)

@xhayper xhayper added the enhancement New feature or request label May 27, 2025
return;
}

editor.statusBarItem.text = "$(search-refresh) Reconnect to Discord Gateway";4
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

;4

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh damn… I was wondering what the emoji comment means.
Took some time until I understood 😅

@MrMinemeet MrMinemeet force-pushed the av/setting-suppress-RPC_COULD_NOT_CONNECT branch from 508587f to 9149faf Compare May 27, 2025 17:04
Useful if Discord is not permanently running (e.g. no auto start-up)
@MrMinemeet MrMinemeet force-pushed the av/setting-suppress-RPC_COULD_NOT_CONNECT branch from 9149faf to a4a5905 Compare May 27, 2025 17:27
@MrMinemeet
Copy link
Contributor Author

The change in the controller.ts shrunk a bit because of some other refactoring

@MrMinemeet MrMinemeet requested a review from xhayper May 27, 2025 17:29
@xhayper
Copy link
Collaborator

xhayper commented May 27, 2025

It shrunk quite a lot!

@xhayper xhayper merged commit 7f17204 into leonardssh:main May 27, 2025
1 check passed
@MrMinemeet MrMinemeet deleted the av/setting-suppress-RPC_COULD_NOT_CONNECT branch May 27, 2025 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants