Skip to content

Added support for automatically opening a file after it is downloaded in desktop mode. #4194#8517

Merged
akshay-joshi merged 2 commits intopgadmin-org:masterfrom
pravesh-sharma:GH-4194
Mar 10, 2025
Merged

Added support for automatically opening a file after it is downloaded in desktop mode. #4194#8517
akshay-joshi merged 2 commits intopgadmin-org:masterfrom
pravesh-sharma:GH-4194

Conversation

@pravesh-sharma
Copy link
Copy Markdown
Contributor

No description provided.

@pravesh-sharma pravesh-sharma force-pushed the GH-4194 branch 2 times, most recently from 7af0bc5 to b3e70ca Compare March 4, 2025 05:59
Copy link
Copy Markdown
Contributor

@akshay-joshi akshay-joshi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Disable the preferences option for File Downloads in server mode.
  2. Update the documentation and screenshots.

@anthonydb
Copy link
Copy Markdown
Contributor

@akshay-joshi Hi, is this functionality going to be enabled by default? If so, I would suggest it be shut off by default. Automatically opening CSV files with Excel, for example, can destroy data. Any US Zip code such as 07031 is automatically converted by Excel to 7031. Unsuspecting users will not know this. Thank you for considering.

@akshay-joshi
Copy link
Copy Markdown
Contributor

@akshay-joshi Hi, is this functionality going to be enabled by default? If so, I would suggest it be shut off by default. Automatically opening CSV files with Excel, for example, can destroy data. Any US Zip code such as 07031 is automatically converted by Excel to 7031. Unsuspecting users will not know this. Thank you for considering.

By Default this setting will be disabled.

Copy link
Copy Markdown
Contributor

@akshay-joshi akshay-joshi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Take the screenshot again with the proper help message. The help message looks old.

@akshay-joshi akshay-joshi merged commit 73b437c into pgadmin-org:master Mar 10, 2025
24 of 32 checks passed
@pravesh-sharma pravesh-sharma deleted the GH-4194 branch May 26, 2025 14:44
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.

3 participants