Skip to content

Fix protocol converter crash on EVALUATE frame define#328

Merged
SukuWc merged 2 commits intomasterfrom
fix/evaluate-frame-protocol-converter
Mar 2, 2026
Merged

Fix protocol converter crash on EVALUATE frame define#328
SukuWc merged 2 commits intomasterfrom
fix/evaluate-frame-protocol-converter

Conversation

@SukuWc
Copy link
Member

@SukuWc SukuWc commented Mar 2, 2026

Summary

  • Rename GRID_CLASS_EVALUATE_ELEMENT_frame to GRID_CLASS_EVALUATE_frame_internal to align with the _frame_start/_frame_end naming convention used by other classes
  • The old name caused protocol_converter.py to match it as a class param and crash with ValueError when trying to cast ".." to int
  • Update the protocol converter workflow to run on all branches (not just master), with deploy and documentation steps gated to master only
  • Remove unused Discord webhook steps from the workflow

🤖 Generated with Claude Code

SukuWc and others added 2 commits March 2, 2026 12:01
Remove unused Discord webhooks. Strip branch filter so the converter
runs on every push, but gate the deploy and documentation steps behind
a master-only check.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…_internal

Align with the _frame_start/_frame_end naming convention used by other
classes. The previous name caused protocol_converter.py to match it as
a class param and crash trying to cast ".." to int.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@SukuWc SukuWc merged commit 9fa533f into master Mar 2, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in Firmware Roadmap Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant