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

File downloads do not work #2

Open
faveoled opened this issue Nov 16, 2024 · 4 comments
Open

File downloads do not work #2

faveoled opened this issue Nov 16, 2024 · 4 comments

Comments

@faveoled
Copy link
Contributor

E.g. "Download image" (ERD diagram view), "Save results to file" (query output window view) fail silently.
The file picker allows specifying the file name though.

@razzeee
Copy link
Member

razzeee commented Nov 18, 2024

Ideally, the app/electron would just use the file portal. Which it should try to do according to https://github.com/pgadmin-org/pgadmin4/blob/9645bb391170a0cb7033bb6e5f38fee192c7b4c6/web/pgadmin/misc/file_manager/static/js/FileManagerModule.jsx#L114 - not sure how exactly it works, but are we not using the runtime they provide but end up on the browser path?

https://github.com/pgadmin-org/pgadmin4/blob/master/runtime/package.json seems to be the part of the project relevant for wrapping everything in electron

@razzeee
Copy link
Member

razzeee commented Nov 19, 2024

I fail at getting pgadmin to build and run locally - it seems, like even the normal shipped (deb or whatever) is also not using the filepicker portal. At least in gnomes case, it's still opening the old interface.

@razzeee
Copy link
Member

razzeee commented Nov 19, 2024

So from playing with https://github.com/electron/electron-quick-start it seems like:

  1. It does the correct thing, when you just use a <input="file" /> html tag
  2. It uses the wrong window, if you call the dialog api https://www.electronjs.org/docs/latest/api/dialog

Ideally pgadmin rewrites to use the former.

@razzeee
Copy link
Member

razzeee commented Nov 19, 2024

Okay, this might actually just be the electron version and the revert of the file portal changes?

It seems to work fine with quickstart in all cases, if I pin it to 31.2.1

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