Skip to content

Enabled large file downloads for desktop users within the query tool. #3369#8753

Merged
akshay-joshi merged 2 commits into
pgadmin-org:masterfrom
adityatoshniwal:3369-download-large-files
May 14, 2025
Merged

Enabled large file downloads for desktop users within the query tool. #3369#8753
akshay-joshi merged 2 commits into
pgadmin-org:masterfrom
adityatoshniwal:3369-download-large-files

Conversation

@adityatoshniwal

Copy link
Copy Markdown
Contributor

No description provided.

@akshay-joshi akshay-joshi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • Remove the "nested block is redundant" code smell at line no 604 in ERDTool.jsx. It's not in your PR, but as you have changed ERDTool.jsx.

  • Fixed a few SonarQube issues in ResultSet.jsx and FileManager.spec.js file (If possible) it's not from your changes.

Comment thread runtime/src/js/pgadmin.js Outdated
@adityatoshniwal adityatoshniwal force-pushed the 3369-download-large-files branch from 810cb64 to cd980c6 Compare May 14, 2025 09:55
@adityatoshniwal

Copy link
Copy Markdown
Contributor Author
  • Remove the "nested block is redundant" code smell at line no 604 in ERDTool.jsx. It's not in your PR, but as you have changed ERDTool.jsx.
  • Fixed a few SonarQube issues in ResultSet.jsx and FileManager.spec.js file (If possible) it's not from your changes.

Fixed.

@akshay-joshi akshay-joshi merged commit 126e1fb into pgadmin-org:master May 14, 2025
32 checks passed
@adityatoshniwal adityatoshniwal deleted the 3369-download-large-files branch May 14, 2025 12:46
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.

2 participants