Re: [pgAdmin4]: Webpacking of static JS/CSS

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(at)pgadmin(dot)org> wrote:

> Thanks, applied.
>
> On Thu, Jul 20, 2017 at 1:35 PM, Surinder Kumar <
> surinder(dot)kumar(at)enterprisedb(dot)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(dot)kumar(at)enterprisedb(dot)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
>