Re: [pgAdmin4]: Webpacking of static JS/CSS - Mailing list pgadmin-hackers

On Thu, Jul 27, 2017 at 10:22 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

Hi Surinder,

'delete_template_files.patch' is applied any more.

Please send the rebased patches.

I have also noticed in the 'move_js_files_to_static_directory.patch', couple of modules have two definitions for 'module_use_template_javascript' method.

Also - 'module_paths.json' is auto generated, please don't include that in the patch.

-- Thanks, Ashesh 

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company

http://www.linkedin.com/in/asheshvashi

On Tue, Jul 25, 2017 at 3:56 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:

Hi,

Please find rebased patches and review.

On Sat, Jul 22, 2017 at 1:03 AM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:

Hi,

Following are the changes in patches attached:

1) Move Javascripts from Templates directory to Static for following modules:

 - About

 - Browser nodes

 - Dashboard

 - FileManager

 - Vendor/snap.svg

 - Preferences

 - Settings

 - Backup

 - Datagrid

 - Debugger

 - Sqleditor

 - Grant Wizard

 - Import & Export

 - Maintenance

 - Restore and

 - User Management

2) Generate module JS path references dynamically/ Remove manually written paths.

Added a new file 'module_paths.json' which will update with absolute paths to module JS defined in every modules > __init__.py > 'def get_own_javascripts'.

When Flask service is started, it accesses the all module javascripts using `current_app.javascripts` which is called inside an route.

This call to route is made using Flask's `test_client` and thus those paths are written to `module_paths.json`(in javascript_bundler.py) which is used by webpack.shim.js file.

Three patches:

1. Move JS files to static directory

2. Deleted template JS

3. Generate JS paths before app starts

These patches needs to be committed separately as deleted files includes in the commit that makes difficult to look for changes through `git log <commit-has>`

Please review.

Thanks,

Surinder

On Thu, Jul 20, 2017 at 6:08 PM, Dave Page <dpage@pgadmin.org> wrote:

Thanks, applied.

On Thu, Jul 20, 2017 at 1:35 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:

Hi Dave,

Can you please review this patch please? I guess it was missed as i send 2 patches back to back in same email chain.

Please find inline detailed description of issue.

On Thu, Jul 20, 2017 at 12:37 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:

Hi

The loading icon image is used in FileManager and it is still referenced from vendor `aciTree`(which is removed) so it gives 404 not found when FileManager is opened from Query tool/Backup utility.

Please find attached patch and review.

Thanks,

Surinder Kumar

--

Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company