Re: [pgAdmin][RM7313] Port file manager to ReactJS - Mailing list pgadmin-hackers
Hi Aditya
Following are the review comments:
GUI:
- By default files/folders are not visible.
- The select button should be disabled until the file/folder is not selected, to reproduce open the dialog select any file, and then change the file format, the select button should be disabled. In one more scenario open the Restore dialog and select any file in the file manager dialog, and click on the 'Delete' option, Select button is enabled and if we click on its file name 'null' is displayed.
- The select folder should not have the 'File Format' option, check the existing behavior. You can check Preferences -> Path -> Binary Path.
- The file should save/create with an extension, open the query tool, and try to save the SQL with file format as 'sql'.
- The save button should be enabled if select the existing file in the Save file dialog. Open the Query tool and type some query, try to save it, and select the existing file, Save button is disabled.
- Storage Manager
- Download, Rename, Delete button should be disabled if no file is selected. Reproduction step, open storage manager dialog, and change the file format to 'All files'. If already it is 'All files' then change to something else and again select 'All files'.
- File drop should be applied only to the marked area not the complete storage/file manager dialog. Currently, the drop icon is not changing if the user tries to drop the file on the rest of the area.
- Delete file/folder option, we should ask for confirmation before deleting the file or folder directly.
- Uploading the same file multiple times shows the same name on the right side with multiple entries, but on disk, there is only one file uploaded. Users should be able to upload the same file multiple times. Thoughts?
- Uploading the same file 2 times and trying to delete the file using the delete button on the right-side panel. it throws an error on the browser.
- Create one folder ex: 'Testing Folder' select that and upload the same file 2 times, one file is uploaded in the same directory and the second one in the parent directory with the folder name prefixed to the file name.
- Create two folders with the same name, as user experience, it shows both the folder has been created without showing any error, but when reopening the dialog or just changing the File format name of the one folder is changed to 'Untitled Folder'. We should throw an error 'Folder already exists.
- Import/Export Serevrs
- Export: Create file dialog should have the primary button label 'Create'.
- Export: File should be saved with extension ".json"
- The file format is JSON, and the dialog shows other files as well. In short JSON filter is not applied, if the file manager dialog opens with 'All files' last time.
Code:
- Remove print statement from 'config.py'
On Wed, Jul 13, 2022 at 6:15 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,Attached is the patch to:
- Port the existing file dialog/storage manager to ReactJS and major UI improvements.
- Use the native OS file dialog when pgAdmin runs in desktop mode.
- Allow users to search files in the current folder. Fixes #7389.
- Fix a bug where save Dialog can't create a new folder. Fixed #7524.
- Allow users to delete files/folders. Fixes #4607.
- Add JS test cases and doc updates.
Please review.--Thanks,Aditya ToshniwalpgAdmin Hacker | Software Architect | edbpostgres.com"Don't Complain about Heat, Plant a TREE"
pgadmin-hackers by date: