8000 Give the fallback names of control types and the { name, type } objects one place each by dominion525 · Pull Request #105 · dominion525/jetdb · GitHub
[go: up one dir, main page]

Skip to content

Give the fallback names of control types and the { name, type } objects one place each - #105

Merged
dominion525 merged 2 commits into
mainfrom
refactor/shared-labels
Oct 21, 2026
Merged

dominion525 merged 2 commits into
mainfrom
refactor/shared-labels

Conversation

@dominion525
Copy link
Copy Markdown
Owner

Summary

  • Add control_type_label to the library: the name of a control type, or its code as 0x.... for one jetdb does not know, as BlobProperty::display_name gives a property. The CLI's form controls and form props, jetdb-wasm and its tests use it instead of each writing the same fallback
  • Make the { name, type } objects of queries, modules and forms in jetdb-wasm with one function, name_and_type, keeping the Rust structs Query, Module and Form, whose types hold different kinds of values
  • The output stays the same

🤖 Generated with Claude Code

dominion525 and others added 2 commits October 21, 2026 09:17
…does not know, from the library

- The CLI's form controls and form props, jetdb-wasm and its tests wrote the same fallback in five places
- Add control_type_label to the library, as BlobProperty::display_name gives a property, and use it in all five; the output stays the same

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- queries, modules and forms wrote the same code for the object of each query, module, and form or report
- Add name_and_type to js.rs and use it in all three, keeping the Rust structs Query, Module and Form, whose types hold different kinds of values; the output stays the same

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@dominion525
dominion525 merged commit 1ebc35d into main Oct 21, 2026
10 checks passed
@dominion525
dominion525 deleted the refactor/shared-labels branch October 21, 2026 00:52
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.

1 participant

0