Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UNIQUE constraint error upon repeated type conversion #5

Closed
alexmojaki opened this issue Jun 12, 2023 · 1 comment
Closed

UNIQUE constraint error upon repeated type conversion #5

alexmojaki opened this issue Jun 12, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@alexmojaki
Copy link
Contributor

When I change column type twice in a row, I get UNIQUE constraint failed: _grist_Tables_column.id failed as it tries to add the helper columns that the logs say were cleaned up after the previous transform:

Screenshot from 2023-06-12 17-28-41

@paulfitz paulfitz added the bug Something isn't working label Jun 12, 2023
paulfitz added a commit to gristlabs/grist-core that referenced this issue Jul 11, 2023
Grist by default uses node-sqlite3 to manipulate data in an
SQLite database. If a single parameter is passed to `run`
and it is a list, the list is unpacked and its contents treated
as the actual parameters. In grist-static, we use other SQLite
interfaces that don't have that automatic unpacking. Most
calls like this have been removed from Grist, but at least one
was missed, and was causing symptoms such as
gristlabs/grist-static#5

This change should make no difference to regular Grist, but
resolves the grist-static problems.
paulfitz added a commit to gristlabs/grist-core that referenced this issue Jul 11, 2023
Grist by default uses node-sqlite3 to manipulate data in an
SQLite database. If a single parameter is passed to `run`
and it is a list, the list is unpacked and its contents treated
as the actual parameters. In grist-static, we use other SQLite
interfaces that don't have that automatic unpacking. Most
calls like this have been removed from Grist, but at least one
was missed, and was causing symptoms such as
gristlabs/grist-static#5

This change should make no difference to regular Grist, but
resolves the grist-static problems.
@paulfitz
Copy link
Member

Should have been fixed by gristlabs/grist-core#566, will reach grist-static soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants