Skip to content

Formate für die Antworten der Chemie-Widgets (#5) - #6

Open
jojohoch wants to merge 1 commit into
mainfrom
chemistry-widget-formats
Open

jojohoch wants to merge 1 commit into
mainfrom
chemistry-widget-formats

Conversation

@jojohoch

Copy link
Copy Markdown

Umsetzung des Vorschlags aus #5: zwei neue Werte für format, je einer für die Antworten der beiden Chemie-Widgets.

Format Typ Wert
CHEM_ELEMENT_SYMBOLS STRING der state des Periodensystems: die Kürzel der gewählten Elemente, durch Leerzeichen getrennt, z. B. H He Li (widget-docs/pse.qmd, „State“)
CHEM_MOLECULE JSON der state des Molekül-Editors: ein JSON-Objekt als String, mit dem Bild des letzten Bearbeitungsstands in asImage (widget-docs/molecule.qmd, „State“)

Die Namen sind ein Vorschlag, die Entscheidung liegt bei dir.

Geändert

  • variable-info.schema.json: beide Werte im Enum von format, $id auf verona-variable-info@2.1, wie bei 1.4 im selben Schritt wie die Änderung
  • variable-info.interface.ts: beide Werte im Typ von format
  • Tests: je ein gültiger Fall pro Format, dazu ein ungültiger mit der kleingeschriebenen Schreibweise

Nicht geändert, weil das bisher im Veröffentlichungsschritt geschah: package_npm.json (Version) und die Beschreibung der Formate auf der Doku-Seite (editor-docs/_variables.qmd in verona-interfaces.github.io, die noch die kleingeschriebenen Werte aus 1.x zeigt).

Geprüft: npm run test_schema läuft mit 8 von 8 Fällen grün. Die neuen gültigen Fälle scheitern am Schema von 2.0 und bestehen mit diesem. Der Typ in variable-info.interface.ts kompiliert. Zum Lauf: npm ci bricht auf main ab, weil package-lock.json nicht zur package.json passt (Missing: encoding@0.1.13). Ohne Lockfile kommt TypeScript 6, an dem die Jest-Konfiguration scheitert, deshalb lief der Test mit TypeScript 5. Den Lockfile habe ich nicht angefasst.

Bezug: iqb-berlin/verona-modules-aspect#1471 (aspect trägt die Formate in seine VariableInfo ein), iqb-berlin/responses#121 (Anzeigetexte).

🤖 Generated with Claude Code

CHEM_ELEMENT_SYMBOLS for the periodic table (the selected element
symbols, separated by spaces) and CHEM_MOLECULE for the molecule
editor (its state object as a JSON string). See issue #5.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

This branch has not been deployed

No deployments
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