Skip to content

fix: sign upgrade, drone GUI sync, manual link guard (NEW-145, 189, 192)#63

Open
ximaks00-hue wants to merge 1 commit into
akki697222:dev-MC1.20from
ximaks00-hue:fix/new-145-189-192-ui-server
Open

fix: sign upgrade, drone GUI sync, manual link guard (NEW-145, 189, 192)#63
ximaks00-hue wants to merge 1 commit into
akki697222:dev-MC1.20from
ximaks00-hue:fix/new-145-189-192-ui-server

Conversation

@ximaks00-hue

Copy link
Copy Markdown
Contributor

Summary

  • NEW-145: Sign upgrade writes via SignText API; returns plain strings
  • NEW-189: Drone running DataSlot uses setRunning not start()/stop() on client
  • NEW-192: skip openUrl on malformed URL

Test plan

  • Robot sign upgrade modifies sign text
  • Drone GUI open does not stop machine
  • Bad manual http link does not NPE client

…145, NEW-189, NEW-192)

- UpgradeSign.setValue now writes via SignText.setMessage/sign.setText instead of copying into a read-only Component snapshot; success returns plain strings (NEW-145).

- Drone menu running DataSlot calls setRunning for display sync only; start()/stop() on the client could halt the machine (NEW-189).

- LinkSegment skips Util.openUrl when URL parsing fails, avoiding client NPE (NEW-192).
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