Skip to content

Signature fixes for some built-in nodes, HttpJson moved to JsonHttp#129

Merged
GaijinKa merged 2 commits intomeetecho:mainfrom
b3by:fixes/nodes
Feb 25, 2026
Merged

Signature fixes for some built-in nodes, HttpJson moved to JsonHttp#129
GaijinKa merged 2 commits intomeetecho:mainfrom
b3by:fixes/nodes

Conversation

@b3by
Copy link
Collaborator

@b3by b3by commented Feb 25, 2026

Description

This PR fixes the class name of the HttpJson node, making it aligned with its containing folder, to JsonHttp. It also fixes some signatures for messages in this very node and in the JsonWebsocket node.

PR type
Select all the labels that apply to this PR.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test update
  • Build/CI configuration change
  • Other (please describe):

Key modifications and changes

  • Changed the name of HttpJson node to JsonHttp
  • Message type annotation fixed for JsonHttp and JsonWebsocket

@b3by b3by requested a review from GaijinKa February 25, 2026 09:49
@b3by b3by added type:bug Something isn't working priority:low labels Feb 25, 2026
@GaijinKa GaijinKa merged commit 832cfc9 into meetecho:main Feb 25, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority:low type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants