fix(tabular-editor): disable nested-field parsing#1164
Merged
Conversation
…g watcher, non-fatal PapaParse warnings, and disable nested-field parsing Replace the watch([tableRef, schemaFields]) + initializeTable() combo with a self-stopping watchEffect that initializes the Tabulator table exactly once. PapaParse warnings (e.g. ragged rows) are now treated as non-fatal: the parsed data is still loaded into the table and the warnings are logged. Only the PapaParse error callback or a synchronous exception falls back to an empty table. Disable Tabulator's nestedFieldSeparator so schema field names containing dots (e.g. note_c2.1) are treated as flat keys. Without this, Tabulator tries to traverse dotted field names as object paths and can throw "Cannot assign to read only property" when the parent value is a primitive string.
ThibaudDauce
approved these changes
Jul 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replace the watch([tableRef, schemaFields]) + initializeTable() combo with a self-stopping watchEffect that initializes the Tabulator table exactly once.
PapaParse warnings (e.g. ragged rows) are now logged as warning: the parsed data is still loaded into the table. Only the PapaParse error callback or a synchronous exception falls back to an empty table.
Disable Tabulator's nestedFieldSeparator so schema field names containing dots (e.g. note_c2.1) are treated as flat keys.
Without this, Tabulator tries to traverse dotted field names as object paths and can throw "Cannot assign to read only property" when the parent value is a primitive string.