Hello Rahul,
Found below issues:
1) The data grid is not fetching more than 1000 records now, gives a console error:
sqleditor.js:1264 Uncaught TypeError: self.enable_disable_download_btn is not a function
at child.fetch_next (sqleditor.js:1264)
2) The sqleditor test cases are failing.
3) Fix pep8 issues.
4) Fix linter issues.
5) Please check the doc changes again, it's not clear to me.
I didn't check but make sure the GUI tests for the sqleditor runs fine since the behaviour has changed now.
Hi Aditya
Can you please review this patch?
Hi Hackers,
Please find the updated patch below.
Hi Hackers,
Please find the attached patch which resolves the issue of macros query results download, have used async cursor to achieve this functionality, where for downloading the results, cursor is scrolled back to 0 and end of the records, and reset again while user scrolling on Data Output table.
QA/Reviewer needs to observe below issues if it occurs:
- If records are more like 5000 or 10000, try to fetch records by scrolling at least (2000), now save the results by clicking the download button, it should save the file, and now try scrolling again, the data should be shown continuously and not any abrupt end or unexpected records order.
- Also, the download button is now "Save results to CSV/TXT" where it will only get enabled when there are valid records in the Data Output.
Also a minor fix of the add folder icon issue is also added into this patch.
--
Rahul Shirsat
Senior Software Engineer | EnterpriseDB Corporation.
--
Rahul Shirsat
Software Engineer | EnterpriseDB Corporation.
--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Principal Software Architect
EDB PostgresMobile: +91 976-788-8246
--
Thanks,Aditya Toshniwal
"Don't Complain about Heat, Plant a TREE"
