Обсуждение: ESLINT: On pgAdmin static javascripts

Поиск
Список
Период
Сортировка

ESLINT: On pgAdmin static javascripts

От
Ashesh Vashi
Дата:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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

Вложения

Re: ESLINT: On pgAdmin static javascripts

От
Dave Page
Дата:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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

Re: ESLINT: On pgAdmin static javascripts

От
Ashesh Vashi
Дата:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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

Re: ESLINT: On pgAdmin static javascripts

От
Dave Page
Дата:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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

Re: ESLINT: On pgAdmin static javascripts

От
Ashesh Vashi
Дата:

On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
:-)

-- Thanks, Ashesh 
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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

Re: ESLINT: On pgAdmin static javascripts

От
Murtuza Zabuawala
Дата:
​Hi Ashesh,

I'm not able to apply patch :( 

murtuza@debian:~/projects/pgadmin4$ git apply ~/Desktop/*.patch
error: patch failed: web/pgadmin/misc/file_manager/static/js/utility.js:10
error: web/pgadmin/misc/file_manager/static/js/utility.js: patch does not apply

 murtuza@debain:~/projects/pgadmin4$ git status | grep '\.rej' | wc -l
      55

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Mon, Jan 8, 2018 at 9:11 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
:-)

-- Thanks, Ashesh 
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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


Re: ESLINT: On pgAdmin static javascripts

От
Ashesh Vashi
Дата:
Please try rest of the patches...
They are independent of each other.

-- Thanks, Ashesh

On Jan 8, 2018 23:45, "Murtuza Zabuawala" <murtuza.zabuawala@enterprisedb.com> wrote:
​Hi Ashesh,

I'm not able to apply patch :( 

murtuza@debian:~/projects/pgadmin4$ git apply ~/Desktop/*.patch
error: patch failed: web/pgadmin/misc/file_manager/static/js/utility.js:10
error: web/pgadmin/misc/file_manager/static/js/utility.js: patch does not apply

 murtuza@debain:~/projects/pgadmin4$ git status | grep '\.rej' | wc -l
      55

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Mon, Jan 8, 2018 at 9:11 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
:-)

-- Thanks, Ashesh 
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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


Re: ESLINT: On pgAdmin static javascripts

От
Ashesh Vashi
Дата:


On Tue, Jan 9, 2018 at 12:28 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Please try rest of the patches...
They are independent of each other.

-- Thanks, Ashesh

On Jan 8, 2018 23:45, "Murtuza Zabuawala" <murtuza.zabuawala@enterprisedb.com> wrote:
​Hi Ashesh,

I'm not able to apply patch :( 

murtuza@debian:~/projects/pgadmin4$ git apply ~/Desktop/*.patch
error: patch failed: web/pgadmin/misc/file_manager/static/js/utility.js:10
error: web/pgadmin/misc/file_manager/static/js/utility.js: patch does not apply

 murtuza@debain:~/projects/pgadmin4$ git status | grep '\.rej' | wc -l
      55
Please find the updated patch for "Browser-specific javascript files", which was having diff for 'web/pgadmin/misc/file_manager/static/js/utility.js'.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Mon, Jan 8, 2018 at 9:11 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
:-)

-- Thanks, Ashesh 
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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



Вложения

Re: ESLINT: On pgAdmin static javascripts

От
Murtuza Zabuawala
Дата:
All patches looks good to me except one issue as shown in screenshot which I observed while opening Preferences dialog subsequently.

--
​ Murtuza​

On Tue, Jan 9, 2018 at 1:19 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:


On Tue, Jan 9, 2018 at 12:28 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Please try rest of the patches...
They are independent of each other.

-- Thanks, Ashesh

On Jan 8, 2018 23:45, "Murtuza Zabuawala" <murtuza.zabuawala@enterprisedb.com> wrote:
​Hi Ashesh,

I'm not able to apply patch :( 

murtuza@debian:~/projects/pgadmin4$ git apply ~/Desktop/*.patch
error: patch failed: web/pgadmin/misc/file_manager/static/js/utility.js:10
error: web/pgadmin/misc/file_manager/static/js/utility.js: patch does not apply

 murtuza@debain:~/projects/pgadmin4$ git status | grep '\.rej' | wc -l
      55
Please find the updated patch for "Browser-specific javascript files", which was having diff for 'web/pgadmin/misc/file_manager/static/js/utility.js'.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Mon, Jan 8, 2018 at 9:11 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
:-)

-- Thanks, Ashesh 
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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




Вложения

Re: ESLINT: On pgAdmin static javascripts

От
Ashesh Vashi
Дата:
Hi Murtuza,

On Tue, Jan 9, 2018 at 4:08 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
All patches looks good to me except one issue as shown in screenshot which I observed while opening Preferences dialog subsequently.
I was able to reproduce the issue on 'master' branch without these patches too.
I have fixed the same issue on 'WEBPACK_DEVEL2' branch.

Dependent libraries are using different versions of jQuery, and hence loading two differct jQuery in the webpack.
And, that's causing the issue.

We can take of the issue in a separate patch.

-- Thanks, Ashesh 

--
​ Murtuza​

On Tue, Jan 9, 2018 at 1:19 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:


On Tue, Jan 9, 2018 at 12:28 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Please try rest of the patches...
They are independent of each other.

-- Thanks, Ashesh

On Jan 8, 2018 23:45, "Murtuza Zabuawala" <murtuza.zabuawala@enterprisedb.com> wrote:
​Hi Ashesh,

I'm not able to apply patch :( 

murtuza@debian:~/projects/pgadmin4$ git apply ~/Desktop/*.patch
error: patch failed: web/pgadmin/misc/file_manager/static/js/utility.js:10
error: web/pgadmin/misc/file_manager/static/js/utility.js: patch does not apply

 murtuza@debain:~/projects/pgadmin4$ git status | grep '\.rej' | wc -l
      55
Please find the updated patch for "Browser-specific javascript files", which was having diff for 'web/pgadmin/misc/file_manager/static/js/utility.js'.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Mon, Jan 8, 2018 at 9:11 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
:-)

-- Thanks, Ashesh 
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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





Re: ESLINT: On pgAdmin static javascripts

От
Ashesh Vashi
Дата:
On Thu, Jan 11, 2018 at 11:16 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Murtuza,

On Tue, Jan 9, 2018 at 4:08 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
All patches looks good to me except one issue as shown in screenshot which I observed while opening Preferences dialog subsequently.
I was able to reproduce the issue on 'master' branch without these patches too.
I have fixed the same issue on 'WEBPACK_DEVEL2' branch.

Dependent libraries are using different versions of jQuery, and hence loading two differct jQuery in the webpack.
And, that's causing the issue.

We can take of the issue in a separate patch.
Committed these patches.

Fahar,

You may want to be on top of this as this commit has touched all the modules.

-- Thanks, Ashesh

-- Thanks, Ashesh 

--
​ Murtuza​

On Tue, Jan 9, 2018 at 1:19 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:


On Tue, Jan 9, 2018 at 12:28 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Please try rest of the patches...
They are independent of each other.

-- Thanks, Ashesh

On Jan 8, 2018 23:45, "Murtuza Zabuawala" <murtuza.zabuawala@enterprisedb.com> wrote:
​Hi Ashesh,

I'm not able to apply patch :( 

murtuza@debian:~/projects/pgadmin4$ git apply ~/Desktop/*.patch
error: patch failed: web/pgadmin/misc/file_manager/static/js/utility.js:10
error: web/pgadmin/misc/file_manager/static/js/utility.js: patch does not apply

 murtuza@debain:~/projects/pgadmin4$ git status | grep '\.rej' | wc -l
      55
Please find the updated patch for "Browser-specific javascript files", which was having diff for 'web/pgadmin/misc/file_manager/static/js/utility.js'.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Mon, Jan 8, 2018 at 9:11 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
:-)

-- Thanks, Ashesh 
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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






Re: ESLINT: On pgAdmin static javascripts

От
Fahar Abbas
Дата:


On Fri, Jan 12, 2018 at 12:35 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Thu, Jan 11, 2018 at 11:16 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Murtuza,

On Tue, Jan 9, 2018 at 4:08 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
All patches looks good to me except one issue as shown in screenshot which I observed while opening Preferences dialog subsequently.
I was able to reproduce the issue on 'master' branch without these patches too.
I have fixed the same issue on 'WEBPACK_DEVEL2' branch.

Dependent libraries are using different versions of jQuery, and hence loading two differct jQuery in the webpack.
And, that's causing the issue.

We can take of the issue in a separate patch.
Committed these patches.

Fahar,

You may want to be on top of this as this commit has touched all the modules.

Sure.
-- Thanks, Ashesh

-- Thanks, Ashesh 

--
​ Murtuza​

On Tue, Jan 9, 2018 at 1:19 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:


On Tue, Jan 9, 2018 at 12:28 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Please try rest of the patches...
They are independent of each other.

-- Thanks, Ashesh

On Jan 8, 2018 23:45, "Murtuza Zabuawala" <murtuza.zabuawala@enterprisedb.com> wrote:
​Hi Ashesh,

I'm not able to apply patch :( 

murtuza@debian:~/projects/pgadmin4$ git apply ~/Desktop/*.patch
error: patch failed: web/pgadmin/misc/file_manager/static/js/utility.js:10
error: web/pgadmin/misc/file_manager/static/js/utility.js: patch does not apply

 murtuza@debain:~/projects/pgadmin4$ git status | grep '\.rej' | wc -l
      55
Please find the updated patch for "Browser-specific javascript files", which was having diff for 'web/pgadmin/misc/file_manager/static/js/utility.js'.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Mon, Jan 8, 2018 at 9:11 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
:-)

-- Thanks, Ashesh 
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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









--
Fahar Abbas
QMG
EnterpriseDB Corporation
Phone Office: +92-51-835-8874
Phone Direct: +92-51-8466803
Mobile: +92-333-5409707
Skype ID: syed.fahar.abbas
Website: www.enterprisedb.com

Re: ESLINT: On pgAdmin static javascripts

От
Fahar Abbas
Дата:
Hi Ashesh,

I have finished smoke verification for all the modules and no issue found so far and currently doing testing in detail.

Kind Regards,


On Mon, Jan 15, 2018 at 10:17 AM, Fahar Abbas <fahar.abbas@enterprisedb.com> wrote:


On Fri, Jan 12, 2018 at 12:35 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Thu, Jan 11, 2018 at 11:16 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Murtuza,

On Tue, Jan 9, 2018 at 4:08 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
All patches looks good to me except one issue as shown in screenshot which I observed while opening Preferences dialog subsequently.
I was able to reproduce the issue on 'master' branch without these patches too.
I have fixed the same issue on 'WEBPACK_DEVEL2' branch.

Dependent libraries are using different versions of jQuery, and hence loading two differct jQuery in the webpack.
And, that's causing the issue.

We can take of the issue in a separate patch.
Committed these patches.

Fahar,

You may want to be on top of this as this commit has touched all the modules.

Sure.
-- Thanks, Ashesh

-- Thanks, Ashesh 

--
​ Murtuza​

On Tue, Jan 9, 2018 at 1:19 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:


On Tue, Jan 9, 2018 at 12:28 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Please try rest of the patches...
They are independent of each other.

-- Thanks, Ashesh

On Jan 8, 2018 23:45, "Murtuza Zabuawala" <murtuza.zabuawala@enterprisedb.com> wrote:
​Hi Ashesh,

I'm not able to apply patch :( 

murtuza@debian:~/projects/pgadmin4$ git apply ~/Desktop/*.patch
error: patch failed: web/pgadmin/misc/file_manager/static/js/utility.js:10
error: web/pgadmin/misc/file_manager/static/js/utility.js: patch does not apply

 murtuza@debain:~/projects/pgadmin4$ git status | grep '\.rej' | wc -l
      55
Please find the updated patch for "Browser-specific javascript files", which was having diff for 'web/pgadmin/misc/file_manager/static/js/utility.js'.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Mon, Jan 8, 2018 at 9:11 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
:-)

-- Thanks, Ashesh 
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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









--
Fahar Abbas
QMG
EnterpriseDB Corporation
Phone Office: +92-51-835-8874
Phone Direct: +92-51-8466803
Mobile: +92-333-5409707
Skype ID: syed.fahar.abbas
Website: www.enterprisedb.com



--
Fahar Abbas
QMG
EnterpriseDB Corporation
Phone Office: +92-51-835-8874
Phone Direct: +92-51-8466803
Mobile: +92-333-5409707
Skype ID: syed.fahar.abbas
Website: www.enterprisedb.com

Re: ESLINT: On pgAdmin static javascripts

От
Ashesh Vashi
Дата:
On Tue, Jan 16, 2018 at 11:10 AM, Fahar Abbas <fahar.abbas@enterprisedb.com> wrote:
Hi Ashesh,

I have finished smoke verification for all the modules and no issue found so far and currently doing testing in detail.
Thanks Fahar for the update!

--
Thanks & Regards,

Ashesh Vashi

Kind Regards,


On Mon, Jan 15, 2018 at 10:17 AM, Fahar Abbas <fahar.abbas@enterprisedb.com> wrote:


On Fri, Jan 12, 2018 at 12:35 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Thu, Jan 11, 2018 at 11:16 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Murtuza,

On Tue, Jan 9, 2018 at 4:08 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
All patches looks good to me except one issue as shown in screenshot which I observed while opening Preferences dialog subsequently.
I was able to reproduce the issue on 'master' branch without these patches too.
I have fixed the same issue on 'WEBPACK_DEVEL2' branch.

Dependent libraries are using different versions of jQuery, and hence loading two differct jQuery in the webpack.
And, that's causing the issue.

We can take of the issue in a separate patch.
Committed these patches.

Fahar,

You may want to be on top of this as this commit has touched all the modules.

Sure.
-- Thanks, Ashesh

-- Thanks, Ashesh 

--
​ Murtuza​

On Tue, Jan 9, 2018 at 1:19 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:


On Tue, Jan 9, 2018 at 12:28 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Please try rest of the patches...
They are independent of each other.

-- Thanks, Ashesh

On Jan 8, 2018 23:45, "Murtuza Zabuawala" <murtuza.zabuawala@enterprisedb.com> wrote:
​Hi Ashesh,

I'm not able to apply patch :( 

murtuza@debian:~/projects/pgadmin4$ git apply ~/Desktop/*.patch
error: patch failed: web/pgadmin/misc/file_manager/static/js/utility.js:10
error: web/pgadmin/misc/file_manager/static/js/utility.js: patch does not apply

 murtuza@debain:~/projects/pgadmin4$ git status | grep '\.rej' | wc -l
      55
Please find the updated patch for "Browser-specific javascript files", which was having diff for 'web/pgadmin/misc/file_manager/static/js/utility.js'.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company



--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Mon, Jan 8, 2018 at 9:11 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I ran both jasmine test framework, and feature tests.
Both are running fine. 

I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. 
Done some manual testing.
It would be helpful, If somebody can help with some more testing, which I may have missed.

Please ask one of the team (other than me) :-)
:-)

-- Thanks, Ashesh 
 

-- Thanks, Ashesh
 

On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave/Team,

I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.
i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>

I had fixed all errors reported by eslint for those files.

I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.
* Browser nodes javascript files
* Browser specific javascript files
* pgAdmin common javascript files
* SQLEditor/DataGrid javascript files
* Tools javascript files
* Miscellaneous modules javascript files

All patches are independent of each other.

Please review it, and share your opinion.
  

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


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




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

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




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

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









--
Fahar Abbas
QMG
EnterpriseDB Corporation
Phone Office: +92-51-835-8874
Phone Direct: +92-51-8466803
Mobile: +92-333-5409707
Skype ID: syed.fahar.abbas
Website: www.enterprisedb.com



--
Fahar Abbas
QMG
EnterpriseDB Corporation
Phone Office: +92-51-835-8874
Phone Direct: +92-51-8466803
Mobile: +92-333-5409707
Skype ID: syed.fahar.abbas
Website: www.enterprisedb.com