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

Error message when trying to export table in XLSX [bug] #9

Closed
guilherme15990 opened this issue Aug 25, 2023 · 7 comments
Closed

Error message when trying to export table in XLSX [bug] #9

guilherme15990 opened this issue Aug 25, 2023 · 7 comments

Comments

@guilherme15990
Copy link

When trying to export a table by going to Share > Export XLSX

bug

Edition: Windows 10 Home Single Language
Version: 22H2
Installed on ‎07/‎12/‎2019
OS Build: 19045.3324
Experience: Windows Feature Experience Pack 1000.19041.1000.0

@paulfitz
Copy link
Member

Thanks for reporting this! I've replicated the problem. Will post when I have a solution.

paulfitz added a commit to paulfitz/grist-core that referenced this issue Aug 27, 2023
Adds a dummy default export to the worker exporter script used
for producing XLSX. This method exists only to make Piscina
happier. With it, Piscina will load this file using a regular
require(), which under Electron will deal fine with Electron's
ASAR app bundle. Without it, Piscina will try fancier methods
that aren't at the time of writing correctly patched to
deal with an ASAR app bundle, and so report that this
file doesn't exist instead of exporting an XLSX file.

I tried various other solutions such as upgrading Electron,
unpacking various files, patching Piscina, and this was
overall the simplest.

See gristlabs/grist-desktop#9
@paulfitz
Copy link
Member

I have a solution in hand (gristlabs/grist-core#646), will work on getting a new release to see if it works for you.

paulfitz added a commit to gristlabs/grist-core that referenced this issue Aug 28, 2023
Adds a dummy default export to the worker exporter script used
for producing XLSX. This method exists only to make Piscina
happier. With it, Piscina will load this file using a regular
require(), which under Electron will deal fine with Electron's
ASAR app bundle. Without it, Piscina will try fancier methods
that aren't at the time of writing correctly patched to
deal with an ASAR app bundle, and so report that this
file doesn't exist instead of exporting an XLSX file.

I tried various other solutions such as upgrading Electron,
unpacking various files, patching Piscina, and this was
overall the simplest.

See gristlabs/grist-desktop#9
@paulfitz
Copy link
Member

Hi @guilherme15990, I made a new release: https://github.com/gristlabs/grist-electron/releases/tag/v0.2.4 - if you have a chance, it'd be great if you could try it out and see if it fixes the problem for you.

@guilherme15990
Copy link
Author

guilherme15990 commented Aug 29, 2023

I installed the new version without uninstalling the old one and I get an error when accessing the database
2023-08-29_12-04

I uninstalled the old version using the HiBit Uninstaller program and the problem persisted

I deleted the database and tried to start a new document, but the problem also persisted

The database writing error also happens in Linux version.

The export function is working, but writing to the database is not

@paulfitz
Copy link
Member

Thanks @guilherme15990, looks like I messed things up badly, sorry. Looking into what has changed with pyodide.

@paulfitz
Copy link
Member

I made a hopefully corrected release: https://github.com/gristlabs/grist-electron/releases/tag/v0.2.5 🤞

@guilherme15990
Copy link
Author

Everything is working now

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

No branches or pull requests

2 participants