Hi Murtuza, On Tue, Jan 9, 2018 at 4:08 PM, Murtuza Zabuawala < murtuza.zabuaw...@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.va...@enterprisedb.com> wrote: > >> >> >> On Tue, Jan 9, 2018 at 12:28 AM, Ashesh Vashi < >> ashesh.va...@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@enterprised >>> b.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 >> <http://www.enterprisedb.com/> >> >> >> *http://www.linkedin.com/in/asheshvashi* >> <http://www.linkedin.com/in/asheshvashi> >> >>> >>>> -- >>>> Regards, >>>> Murtuza Zabuawala >>>> EnterpriseDB: http://www.enterprisedb.com >>>> The Enterprise PostgreSQL Company >>>> >>>> >>>> On Mon, Jan 8, 2018 at 9:11 PM, Ashesh Vashi < >>>> ashesh.va...@enterprisedb.com> wrote: >>>> >>>>> >>>>> On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dp...@pgadmin.org> wrote: >>>>> >>>>>> Hi >>>>>> >>>>>> On Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi < >>>>>> ashesh.va...@enterprisedb.com> wrote: >>>>>> >>>>>>> On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dp...@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.va...@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.enterprisedb.com> >>>>>>>>> >>>>>>>>> >>>>>>>>> *http://www.linkedin.com/in/asheshvashi* >>>>>>>>> <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 >>>>>> >>>>> >>>>> >>>> >> >