[pgadmin-hackers] pgAdmin 4 commit: Fix runtime support under Qt4

2017-06-16 Thread Dave Page
Fix runtime support under Qt4 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=a21dad4c456f62a3581dc3152a3ef988a25ba15e Author: Neel Patel Modified Files -- runtime/TabWindow.cpp | 4 runtime/TabWindow.h | 16 +--

[pgadmin-hackers] pgAdmin 4 commit: Ensure the feature tests use the correct test setting

2017-06-16 Thread Dave Page
Ensure the feature tests use the correct test settings database. Fixes #2486 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=1ee555e4687dcd94cc054738076b4cb78999e81c Author: Khushboo Vashi Modified Files -- web/config.py

Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fixed #2486: Feature tests use SQLITE_PATH instead of TEST_SQLITE_PATH.

2017-06-16 Thread Dave Page
> Thanks, > Khushboo > > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter:

[pgadmin-hackers] pgAdmin 4 commit: Add support for detachable tabs to the runtime.

2017-06-16 Thread Dave Page
Add support for detachable tabs to the runtime. This allows the Query Tool, Debugger and web browser tabs to be moved to different monitors as desired. Fixes #1344 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=0056a94115416c12690f58af3b77b

Re: [pgadmin-hackers] Re: Server side cursor limitations for on demand loading of data in query tool [RM2137] [pgAdmin4]

2017-06-15 Thread Dave Page
Sounds good, thanks. On Thu, Jun 15, 2017 at 9:54 PM, Harshal Dhumal wrote: > Hi > > On Fri, Jun 16, 2017 at 2:07 AM, Dave Page wrote: >> >> Hi >> >> On Thu, Jun 15, 2017 at 9:30 PM, Harshal Dhumal >> wrote: >> > Hi Dave, >> > >>

Re: [pgadmin-hackers] Re: Server side cursor limitations for on demand loading of data in query tool [RM2137] [pgAdmin4]

2017-06-15 Thread Dave Page
Hi On Thu, Jun 15, 2017 at 9:30 PM, Harshal Dhumal wrote: > Hi Dave, > > Please find attached updated patch. > > On Thu, Jun 15, 2017 at 3:58 PM, Dave Page wrote: >> >> Hi >> >> On Wed, Jun 14, 2017 at 11:36 PM, Harshal Dhumal >> wrote: >>&

Re: [pgadmin-hackers][patch] History Detail Pane

2017-06-15 Thread Dave Page
;> >>> Do we understand the issues Dave is mentioning well enough to look into >>> it and tackle it? >>> >>> -- Rob >>> >>> On Wed, Jun 14, 2017 at 8:12 AM, Dave Page wrote: >>>> >>>> Hi, >>>> >>>> Be

[pgadmin-hackers] pgAdmin 4 commit: Fix Yarn tasks for Windows

2017-06-15 Thread Dave Page
Fix Yarn tasks for Windows Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=5d45dee686056663795c0dbea5a310d8950df2e4 Modified Files -- Make-MinGW.bat | 4 ++-- Make.bat

Re: [pgadmin-hackers] Minor typo in web/config.py

2017-06-15 Thread Dave Page
make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers mailing list (pgadmin-hac

[pgadmin-hackers] pgAdmin 4 commit: Fix a typo in the config file

2017-06-15 Thread Dave Page
Fix a typo in the config file Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=827aa72e99364cf78f57d7ac1a906f644c961e5e Author: Johan Van de Wauw Modified Files -- web/config.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [pgadmin-hackers] pgAdmin 4 commit: Use a more sensible name for Query Tool tabs. Fixes #

2017-06-15 Thread Dave Page
Kumar < surinder.ku...@enterprisedb.com> wrote: > Hi Dave, > > Please find attached patch. It should work. > In previous patch I was unaware that i added untracked file to tracked. > > On Thu, Jun 15, 2017 at 4:10 PM, Dave Page wrote: > >> Hi >> >> Wa

[pgadmin-hackers] pgAdmin 4 commit: Fix script options which were inadvertently broken in

2017-06-15 Thread Dave Page
Fix script options which were inadvertently broken in the query tool tab naming patch. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=2a87585665049fb67ca9d51fd949b3bd7d25b077 Author: Surinder Kumar Modified Files -- web/pgadmin/

[pgadmin-hackers] pgAdmin 4 commit: Add double precision[] tests to the query tool.

2017-06-15 Thread Dave Page
Add double precision[] tests to the query tool. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=d70c3003d3fdadfbcfff4f677f25a9f3f35471cd Author: Murtuza Zabuawala Modified Files -- .../feature_tests/pg_datatype_validation_test.py

Re: [pgadmin-hackers] [pgAdmin4][PATCH] Fix handling of double precision[] type

2017-06-15 Thread Dave Page
ala > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > On Thu, Jun 8, 2017 at 7:21 PM, Dave Page wrote: >> >> Thanks, applied. >> >> Can you update the data types feature test to cover this case please? >> >> Thanks. >> >> On Th

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in Grant wizard

2017-06-15 Thread Dave Page
egards, > Murtuza Zabuawala > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/

[pgadmin-hackers] pgAdmin 4 commit: Fix canceling of Grant Wizard on Windows. Fixes #2142

2017-06-15 Thread Dave Page
Fix canceling of Grant Wizard on Windows. Fixes #2142 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=8376d33d775ddb79ecedae93f9d969d8955693f5 Author: Murtuza Zabuawala Modified Files -- .../tools/grant_wizard/templates/grant_wiza

Re: [pgadmin-hackers] pgAdmin 4 commit: Use a more sensible name for Query Tool tabs. Fixes #

2017-06-15 Thread Dave Page
>> callback @ jquery-1.11.2.js:9713 >> >> >> >> >> -- >> *Harshal Dhumal* >> *Sr. Software Engineer* >> >> EnterpriseDB India: http://www.enterprisedb.com >> The Enterprise PostgreSQL Company >> >> On Tue, Jun 13, 2017 at 4:04 PM, Da

Re: [pgadmin-hackers] [pgAdmin4][PATCH] Improvements to Query Results Grid User Experience

2017-06-15 Thread Dave Page
On Fri, Jun 9, 2017 at 9:03 AM, Dave Page wrote: > On Thu, Jun 8, 2017 at 7:31 PM, Sarah McAlear wrote: >>> Thanks. When I run the tests my browser opens in some default size >>> that's always consistent, but doesn't match my normal Chrome sessions, >

Re: [pgadmin-hackers][patch] History Detail Pane

2017-06-14 Thread Dave Page
hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hac

[pgadmin-hackers] pgAdmin 4 commit: Add some more Makefile targets for running different

2017-06-14 Thread Dave Page
Add some more Makefile targets for running different test suites. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=6938fbafb538997e8a37a19827b03ad402572829 Modified Files -- Makefile | 11 ++- 1 file changed, 10 insertions(+)

Re: [pgadmin-hackers] Re: Server side cursor limitations for on demand loading of data in query tool [RM2137] [pgAdmin4]

2017-06-14 Thread Dave Page
t;> viii. explain analyze query with buffers >>> ix. explain analyze query with timing >>> x. auto commit disabled. >>> xi. auto commit enabled. >>> xii. auto rollback enabled. >>> xiii. cancel query. >>> >>> &

Re: [pgadmin-hackers] Declarative partitioning in pgAdmin4

2017-06-14 Thread Dave Page
On Wed, Jun 14, 2017 at 1:06 PM, Ashesh Vashi wrote: > On Wed, Jun 14, 2017 at 1:59 PM, Dave Page wrote: > >> >> >> On Tue, Jun 13, 2017 at 2:59 PM, Akshay Joshi < >> akshay.jo...@enterprisedb.com> wrote: >> >>> Hi All >>> >>

Re: [pgadmin-hackers] [pgAdmin4][Patch][RM_ 2477]: New Line text edit pop up renders off page when the size of the grid exceeds the size of the window

2017-06-14 Thread Dave Page
Thanks, patch applied. On Tue, Jun 13, 2017 at 12:30 PM, Surinder Kumar wrote: > Hi > > On Tue, Jun 13, 2017 at 2:54 PM, Dave Page wrote: >> >> On Mon, Jun 12, 2017 at 5:55 PM, Shirley Wang wrote: >> >>> >> >>> >> >>> On Mon,

[pgadmin-hackers] pgAdmin 4 commit: Ensure text editors render in an appropriate place on

2017-06-14 Thread Dave Page
Ensure text editors render in an appropriate place on the results grid. Fixes #2477 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=fdc9eed51479e0057420f116f4b3b7425a12f377 Author: Surinder Kumar Modified Files -- web/pgadmin/fea

[pgadmin-hackers] pgAdmin 4 commit: Un-break the feature tests. Fixes from Surinder and m

2017-06-14 Thread Dave Page
Un-break the feature tests. Fixes from Surinder and myself. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=2b4a8ed89c9374fa9e0f9c0eb190662979d0bb6b Author: Surinder Kumar Modified Files -- web/pgadmin/feature_tests/copy_selected_

Re: [pgadmin-hackers] [pgAdmin4][Patch][RM_2482]: Query tool/ Edit grid panel should have meaningful title

2017-06-14 Thread Dave Page
Thanks Surinder. That, along with another fix seems to have got the tests working fully again. On Tue, Jun 13, 2017 at 6:37 PM, Surinder Kumar wrote: > Hi > > PFA patch. > > On Tue, Jun 13, 2017 at 9:54 PM, Dave Page wrote: >> >> Hmm, this seems to have broken the fea

[pgadmin-hackers] pgAdmin 4 commit: Fix the JS Bundler test to work when there are multip

2017-06-14 Thread Dave Page
Fix the JS Bundler test to work when there are multiple servers defined in a test run. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=4b846ae4e04efee32156c4def44346a6878de6b0 Author: Joao Pedro De Almeida Pereira Modified Files

Re: [pgadmin-hackers][patch] Correct bundle test that polluted Python modules

2017-06-14 Thread Dave Page
Thanks, that got it. Applied! On Tue, Jun 13, 2017 at 4:51 PM, Joao Pedro De Almeida Pereira wrote: > Hi Hackers, > > Good catch Dave. > > You can find attached the new diff with the correction of the problem above. > > Thanks > Shruti & Joao > > On Tue, J

Re: [pgadmin-hackers] Declarative partitioning in pgAdmin4

2017-06-14 Thread Dave Page
alogue is opened - e.g. if you toggle "Enable Trigger" to disabled and hit OK, then next time you open the dialogue you see the switch in the same position. With Attach, that's not the case - you'll list one or more tables to attach, hit OK, and when you next open the

Re: [pgadmin-hackers] [pgAdmin4][Patch][RM_2482]: Query tool/ Edit grid panel should have meaningful title

2017-06-13 Thread Dave Page
Hmm, this seems to have broken the feature tests. Can you look at that ASAP please? I think it's because the tab text has changed. On Tue, Jun 13, 2017 at 11:34 AM, Dave Page wrote: > Thanks, patch applied. > > On Tue, Jun 13, 2017 at 6:24 AM, Surinder Kumar > wrote: &g

Re: [pgadmin-hackers][patch] Correct bundle test that polluted Python modules

2017-06-13 Thread Dave Page
7;bundle:dev']) File "/Users/dpage/.virtualenvs/pgadmin4/lib/python2.7/site-packages/mock/mock.py", line 947, in assert_called_once_with raise AssertionError(msg) AssertionError: Expected 'call' to be called once. Called 0 times. ---

Re: [pgadmin-hackers] pgAdmin 4 commit: Temporarily disable the Javascript bundler test until

2017-06-13 Thread Dave Page
e second (and I imagine subsequent) servers. Can you take a look at this please? Thanks. On Tue, Jun 13, 2017 at 2:22 PM, Dave Page wrote: > Temporarily disable the Javascript bundler test until it handle more than one > run. > > Branch > -- > master > > Details > -

[pgadmin-hackers] pgAdmin 4 commit: Temporarily disable the Javascript bundler test until

2017-06-13 Thread Dave Page
Temporarily disable the Javascript bundler test until it handle more than one run. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=cb97722fc3826f94f375f5f48807a486383ee3a9 Modified Files -- web/pgadmin/utils/javascript/tests/test_

Re: [pgadmin-hackers] [pgAdmin4] [PATCH] History Tab rewrite in React

2017-06-13 Thread Dave Page
On Tue, Jun 13, 2017 at 11:47 AM, Surinder Kumar wrote: > On Tue, Jun 13, 2017 at 4:05 PM, Dave Page wrote: >> >> On Tue, Jun 13, 2017 at 11:31 AM, Surinder Kumar >> wrote: >> > >> > On Tue, Jun 13, 2017 at 3:56 PM, Dave Page wrote: >> >> &

Re: [pgadmin-hackers] [pgAdmin4] [PATCH] History Tab rewrite in React

2017-06-13 Thread Dave Page
On Tue, Jun 13, 2017 at 11:31 AM, Surinder Kumar wrote: > > On Tue, Jun 13, 2017 at 3:56 PM, Dave Page wrote: >> >> On Tue, Jun 13, 2017 at 11:22 AM, Ashesh Vashi >> wrote: >> > On Tue, Jun 13, 2017 at 2:47 PM, Dave Page wrote: >> >> >> &

Re: [pgadmin-hackers] [pgAdmin4][Patch][RM_2482]: Query tool/ Edit grid panel should have meaningful title

2017-06-13 Thread Dave Page
Thanks > Surinder > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake Enterpris

[pgadmin-hackers] pgAdmin 4 commit: Use a more sensible name for Query Tool tabs. Fixes #

2017-06-13 Thread Dave Page
Use a more sensible name for Query Tool tabs. Fixes #2482 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=57d866f7b70996a82a881b20248c3285ab3a6900 Author: Surinder Kumar Modified Files -- web/pgadmin/browser/templates/browser/js/n

Re: [pgadmin-hackers] [pgAdmin4] [PATCH] History Tab rewrite in React

2017-06-13 Thread Dave Page
On Tue, Jun 13, 2017 at 11:22 AM, Ashesh Vashi wrote: > On Tue, Jun 13, 2017 at 2:47 PM, Dave Page wrote: >> >> And then I find a problem. Sigh. >> >> When running in the desktop runtime, under QtWekKit (the forked, >> updated version that is by far the best of

Re: [pgadmin-hackers] pgAdmin 4 commit: Required mock package for python < 3.3.

2017-06-13 Thread Dave Page
sedb.com> >> >> >> *http://www.linkedin.com/in/asheshvashi* >> <http://www.linkedin.com/in/asheshvashi> >> >> On Tue, Jun 13, 2017 at 3:50 PM, Dave Page wrote: >> >>> Is mock required at runtime, or just for tests? It's already in >&

Re: [pgadmin-hackers] pgAdmin 4 commit: Required mock package for python < 3.3.

2017-06-13 Thread Dave Page
To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers mailing list (pgadmin-h

Re: [pgadmin-hackers] [pgAdmin4] [PATCH] Skip CheckForViewDataTest test based on the DB version

2017-06-13 Thread Dave Page
sion 9.1 of Postgres, per the documentation. We decided to skip this test >> for now for all PG Versions below 9.1 >> >> Thanks >> Shruti & Joao > > > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) >

[pgadmin-hackers] pgAdmin 4 commit: Skip a test that uses the COLLATE option on PG versio

2017-06-13 Thread Dave Page
Skip a test that uses the COLLATE option on PG versions below 9.1 (e.g. Greenplum). Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=a9ce9c97173e904792073597f90ff30c4bfdc1fc Author: Joao Pedro De Almeida Pereira Modified Files --

Re: [pgadmin-hackers] [pgAdmin4][Patch][RM_ 2477]: New Line text edit pop up renders off page when the size of the grid exceeds the size of the window

2017-06-13 Thread Dave Page
>>> >>> I prefer 1 if possible, then 2, finally 3. >>> >>> Shirley, any thoughts? > > > Option 1 would require us choosing something new and free (there are a lot > of webfonts you can get from Google) but would take some time to pick the > a

Re: [pgadmin-hackers] [pgAdmin4][Patch]: Added the shortcut (CTRL + 0) to reset the zoom level in runtime

2017-06-13 Thread Dave Page
Nice work! Committed (along with an update to the docs). Congratulations on your first C++ patch :-) On Tue, Jun 13, 2017 at 7:07 AM, Khushboo Vashi wrote: > Hi, > > On Mon, Jun 12, 2017 at 5:27 PM, Dave Page wrote: >> >> Hi >> >> On Mon, Jun 12, 2017 at

[pgadmin-hackers] pgAdmin 4 commit: Add a shortcut (Ctrl/Cmd + 0 (zero)) to the runtime t

2017-06-13 Thread Dave Page
Add a shortcut (Ctrl/Cmd + 0 (zero)) to the runtime to allow the zoom level to be reset. Fixes #2485 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=3a5fda44e400492bd76811906f5261334d819956 Author: Khushboo Vashi Modified Files -

Re: [pgadmin-hackers] [pgAdmin4] [PATCH] History Tab rewrite in React

2017-06-13 Thread Dave Page
uld you answer my earlier question about the need for the delay in app_starter.py please? Thanks. > On Mon, Jun 12, 2017 at 12:15 PM Dave Page wrote: >> >> To add to that; running the JS tests gives: >> >> ERROR in ./regression/javascript/history/query_history_entry_spe

[pgadmin-hackers] pgAdmin 4 commit: Initial re-vamp of the History tab.

2017-06-13 Thread Dave Page
Initial re-vamp of the History tab. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=1208206bc0eca2d135469258e8a209b983e668be Author: Shruti B Iyer Modified Files -- web/.eslintrc.js | 7 +- web/k

Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fixed RM 2324 - PostGIS datatypes not showing up properly on SQL tab.

2017-06-13 Thread Dave Page
Thanks, applied. On Tue, Jun 13, 2017 at 4:45 AM, Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi, > > On Mon, Jun 12, 2017 at 8:24 PM, Dave Page wrote: > >> Hi Khushboo, >> >> Looks like this patch broke the regressions tests :-(. Can you take

[pgadmin-hackers] pgAdmin 4 commit: Fix the fix for RM2324.

2017-06-13 Thread Dave Page
Fix the fix for RM2324. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=16a15bf9344ed9938cade6eb7090cd7149c4b550 Author: Khushboo Vashi Modified Files -- .../servers/databases/schemas/tables/__init__.py | 18 +++--

Re: [pgadmin-hackers] PG Admin 4

2017-06-13 Thread Dave Page
contain > confidential and legally privileged information. If you are not the > intended recipient, please reply to advise the sender and do not disclose, > copy or redistribute this document or any information contained herein. > > Please consider the environment before printing this email. > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company

Re: [pgadmin-hackers] [pgAdmin4] [PATCH] Translatable Text in Query Results Header

2017-06-12 Thread Dave Page
> > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.

[pgadmin-hackers] pgAdmin 4 commit: Bring React into the tree, and add linting and bundli

2017-06-12 Thread Dave Page
Bring React into the tree, and add linting and bundling framework for the JS etc. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=659eb1c1e821e2ea414dc00d22022b8a2d7c91f7 Author: Shruti B Iyer Modified Files -- .gitignore

[pgadmin-hackers] pgAdmin 4 commit: Add a missing translation call.

2017-06-12 Thread Dave Page
Add a missing translation call. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=f1acad85db4b9bf8a201cd8b6d90c703322a0073 Author: Shruti B Iyer Modified Files -- web/pgadmin/static/js/selection/grid_selector.js | 5 +++-- 1 file cha

[pgadmin-hackers] pgAdmin 4 commit: Add linting support, and, well, lint.

2017-06-12 Thread Dave Page
Add linting support, and, well, lint. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=3f4f8b9e77204a62ecdbf99caf296b6ac39b0f5e Author: Shruti B Iyer Modified Files -- web/.eslintrc.js | 45 ++

Re: [pgadmin-hackers] [pgAdmin4] [PATCH] History Tab rewrite in React

2017-06-12 Thread Dave Page
code 1. info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. Also, while I think of it, why the addition of the delay to app_starter.py? On Mon, Jun 12, 2017 at 5:12 PM, Dave Page wrote: > Hi, > > So 01 and 02 are now committed :-). > > 03 has a c

[pgadmin-hackers] PACKAGERS BEWARE: Build changes required

2017-06-12 Thread Dave Page
loper/packager will need to do this. 2) Web-packing of all the JS and CSS code. This will minimise all the JS/CSS to reduce file transfer sizes, and bundle it together into as few files as possible to minimise client/server round trips. 3) A move towards React to improve UI rendering/generation etc. --

Re: [pgadmin-hackers] [pgAdmin4][Patch][Feature_1535]: Pressing ESC from within a dialog box should act like "Cancel" button

2017-06-12 Thread Dave Page
Thanks, applied. On Mon, Jun 12, 2017 at 3:52 PM, Surinder Kumar wrote: > Hi Dave, > > Please find updated patch. > > On Mon, Jun 12, 2017 at 7:58 PM, Dave Page wrote: >> >> Hi >> >> On Mon, Jun 12, 2017 at 2:03 PM, Surinder Kumar >> wrote: >

[pgadmin-hackers] pgAdmin 4 commit: Teach dialogues about Escape to cancel, Enter to Save

2017-06-12 Thread Dave Page
Teach dialogues about Escape to cancel, Enter to Save/OK, and F1 for help. Fixes #1535 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=af43ccfc07bd1f919015b79ce9c9e7e06d92cf6a Author: Surinder Kumar Modified Files -- .../template

Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fixed RM 2324 - PostGIS datatypes not showing up properly on SQL tab.

2017-06-12 Thread Dave Page
_formatter(tid, data) File "/Users/pivotal/workspace/pgadmin4/web/pgadmin/browser/ server_groups/servers/databases/schemas/tables/__init__.py", line 663, in _columns_formatter column['attlen'] = matchObj.group(1) AttributeError: 'NoneType' object has no attribute

Re: [pgadmin-hackers] [pgAdmin4] [PATCH] History Tab rewrite in React

2017-06-12 Thread Dave Page
t__.py", > line 663, in _columns_formatter > column['attlen'] = matchObj.group(1) > AttributeError: 'NoneType' object has no attribute 'group' > > Was this introduced in a previous patch? Yes, it looks like it. For some reason it's not fa

Re: [pgadmin-hackers] [pgAdmin4][Patch][RM_ 2477]: New Line text edit pop up renders off page when the size of the grid exceeds the size of the window

2017-06-12 Thread Dave Page
r. 3) Choose something completely new and free, and ship it with the app. I prefer 1 if possible, then 2, finally 3. Shirley, any thoughts? > On Mon, Jun 12, 2017 at 6:16 PM, Dave Page wrote: >> >> Hi >> >> Can you update this so that the CSS all follows the standards i

Re: [pgadmin-hackers] [pgAdmin4][Patch][Feature_1535]: Pressing ESC from within a dialog box should act like "Cancel" button

2017-06-12 Thread Dave Page
he: if (panel && panel['_type'] !== 'node_props') return; into the branch for keyCode.ENTER, but that doesn't seem to help. -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Comp

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in Role Membership control

2017-06-12 Thread Dave Page
za Zabuawala > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -

[pgadmin-hackers] pgAdmin 4 commit: Allow editing of the WITH ADMIN option of role member

2017-06-12 Thread Dave Page
Allow editing of the WITH ADMIN option of role membership. Fixes #2086 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=ec5bc23dfdb41da1b5ccd4ab2ab2e6244d8b5ebd Author: Murtuza Zabuawala Modified Files -- .../browser/server_groups/

Re: [pgadmin-hackers] [pgAdmin4][Patch][Feature_1535]: Pressing ESC from within a dialog box should act like "Cancel" button

2017-06-12 Thread Dave Page
Hi Is that the correct patch? Hitting in a comment text area is still causing the dialogue to close. Thanks. On Mon, Jun 12, 2017 at 12:55 PM, Surinder Kumar wrote: > Hi > > Please find rebased patch > > Thanks > > On Mon, Jun 12, 2017 at 5:20 PM, Dave Page wrote: >&g

Re: [pgadmin-hackers] [pgAdmin4][Patch][RM_ 2477]: New Line text edit pop up renders off page when the size of the grid exceeds the size of the window

2017-06-12 Thread Dave Page
gt; file. > > Please find attached patch and review. > > Thanks > Surinder > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Pag

[pgadmin-hackers] pgAdmin 4 commit: Refresh the SQL editor view on resize to ensure the c

2017-06-12 Thread Dave Page
Refresh the SQL editor view on resize to ensure the contents are re-rendered for the new viewport. Fixes #2078 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=3743c6e86ed74b8ae6f4776f27e1555c4ee64e89 Author: Murtuza Zabuawala Modified Files

Re: [pgadmin-hackers] [pgAdmin4][PATCH] CREATE Script do not display properly on panel resize

2017-06-12 Thread Dave Page
> EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blo

Re: [pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue in displaying Trigger SQL

2017-06-12 Thread Dave Page
ters. > RM#2043 > > -- > Regards, > Murtuza Zabuawala > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > > -- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www

[pgadmin-hackers] pgAdmin 4 commit: Properly handle trigger functions with parameters. Fi

2017-06-12 Thread Dave Page
Properly handle trigger functions with parameters. Fixes #2043 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=c435d488853b7b52f33b8d01fe98b5938e6fd835 Author: Murtuza Zabuawala Modified Files -- .../servers/databases/schemas/tabl

Re: [pgadmin-hackers] [pgAdmin4][Patch]: Added the shortcut (CTRL + 0) to reset the zoom level in runtime

2017-06-12 Thread Dave Page
startup value is really what we want. Shouldn't it reset it to 1.0? -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.

Re: [pgadmin-hackers] [pgAdmin4][Patch][RM_2450] : Internal Server Error displayed if click on statistic tab for PG 10 server

2017-06-12 Thread Dave Page
Thanks Atul, patch applied. On Fri, Jun 9, 2017 at 1:34 PM, Atul Sharma wrote: > Hi Dave, > > I have made the template search code to work for 10_plus directory name. > Please review the attached patch > > Regards, > Atul. > > > > On Thu, Jun 8, 2017 at 7:55 PM

[pgadmin-hackers] pgAdmin 4 commit: Fix error on the stats tab with PG10. Also, rename th

2017-06-12 Thread Dave Page
Fix error on the stats tab with PG10. Also, rename the 10.0_plus template directory to 10_plus to match the new versioning. Fixes #2450 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=49cd578816ab45166556fcd46096671130939cc3 Author: Atul Sharm

Re: [pgadmin-hackers] [pgAdmin4][Patch][Feature_1535]: Pressing ESC from within a dialog box should act like "Cancel" button

2017-06-12 Thread Dave Page
> >> PFA patch and review. >> >> >> On Fri, Jun 9, 2017 at 1:47 PM, Dave Page wrote: >>> >>> Hi >>> >>> Ahh, now we have a bug. In the properties dialogues, if you edit a >>> comment, as soon as the Save button gets enabled, hit

[pgadmin-hackers] pgAdmin 4 commit: Fix the PostGIS Datatypes in SQL tab, Create / Update

2017-06-12 Thread Dave Page
Fix the PostGIS Datatypes in SQL tab, Create / Update dialogues for Table, Column, Foreign Table and Type node. Fixes #2324 Note that this doesn't add full support for PostGIS types to the table/column dialogues, which still can't handle the typmod usage. It does bring support to parity with pg

Re: [pgadmin-hackers] [pgAdmin4][Patch]: Fixed RM 2324 - PostGIS datatypes not showing up properly on SQL tab.

2017-06-12 Thread Dave Page
tion with Dave, I have logged RM #2473 > <https://redmine.postgresql.org/issues/2473> to add support for PostGIS > Datatypes. > > Thanks, > Khushboo > > On Tue, Jun 6, 2017 at 6:49 PM, Dave Page wrote: > >> >> >> On Tue, Jun 6, 2017 at 2:15 PM, Khushboo Vashi <

[pgadmin-hackers] pgAdmin 4 commit: Stop ignoring yarn.lock, per discussion with Pivotal

2017-06-12 Thread Dave Page
Stop ignoring yarn.lock, per discussion with Pivotal team. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=59da69087cd32ea9e4996f87bae62c8c31dc632b Author: George Gelashvili Modified Files -- .gitignore| 1 - web/yarn.lock | 0

[pgadmin-hackers] pgAdmin 4 commit: Tune startup retry loop - try 50 times, with a 100ms

2017-06-11 Thread Dave Page
Tune startup retry loop - try 50 times, with a 100ms pause between each attempt. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=dea850b503ccdf270199c52621d36103cfe414c3 Modified Files -- runtime/pgAdmin4.cpp | 6 -- 1 file chan

[pgadmin-hackers] pgAdmin 4 commit: Another minor perf tweak for the runtime.

2017-06-11 Thread Dave Page
Another minor perf tweak for the runtime. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=ef4b8e8c1c563d51641c2eb2f2f81475a00a09d9 Modified Files -- runtime/Server.cpp | 1 + 1 file changed, 1 insertion(+) -- Sent via pgadmin-hac

[pgadmin-hackers] pgAdmin 4 commit: Couple of minor performance tweaks. Tests on my Windo

2017-06-11 Thread Dave Page
Couple of minor performance tweaks. Tests on my Windows VM show a consistent 30% startup time reduction. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=f0d287588ace15dee3e4e9447ec54d9b4fef8491 Modified Files -- runtime/pgAdmin4.c

Re: [pgadmin-hackers] Re: [pgAdmin4][Patch][Feature #1971]: Remember column sizes between executions of the same query in the query tool

2017-06-09 Thread Dave Page
Thanks, applied. On Fri, Jun 9, 2017 at 8:39 AM, Surinder Kumar wrote: > Hi > > As the patch "Improvements to Query Results" > is merged, the columns width in edit grid needs some adjustments as > 'checkboxes' are removed from columns. > > PFA patch. >

[pgadmin-hackers] pgAdmin 4 commit: Fine tune result grid column sizing now we don't have

2017-06-09 Thread Dave Page
Fine tune result grid column sizing now we don't have checkboxes. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=49976b193476c7626c305853c23cbf9a98af68e8 Author: Surinder Kumar Modified Files -- web/pgadmin/static/js/sqleditor_ut

Re: [pgadmin-hackers] [pgAdmin4][Patch][Feature_1535]: Pressing ESC from within a dialog box should act like "Cancel" button

2017-06-09 Thread Dave Page
> Hi > > Please find updated patch. > > Thanks, > Surinder > > On Thu, Jun 8, 2017 at 7:16 PM, Dave Page wrote: >> >> Hi >> >> On Thu, Jun 8, 2017 at 5:38 AM, Surinder Kumar >> wrote: >> > Hi >> > >> > The attribute key fo

Re: [pgadmin-hackers][patch] Changing the ACI tree font to Helvetica

2017-06-09 Thread Dave Page
-- > Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com

[pgadmin-hackers] pgAdmin 4 commit: Use the correct font for the treeview.

2017-06-09 Thread Dave Page
Use the correct font for the treeview. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=ee4a84778457e3ca022be4e2c7f730c98a5e3422 Author: Sarah McAlear Modified Files -- web/pgadmin/static/css/aci_tree.overrides.css | 4 web/pga

Re: [pgadmin-hackers] [pgAdmin4][PATCH] Improvements to Query Results Grid User Experience

2017-06-09 Thread Dave Page
00)" > The issue was still present for us when we ran the app at each of 2fddf750 > and 495a3cedb > Could we move this discussion to a new thread as it doesn't seem related to > these changes? Something in these changes caused it to start manifesting, but I agree the underlying

[pgadmin-hackers] pgAdmin 4 commit: A little leanup of the styling of the edit grid

2017-06-08 Thread Dave Page
A little leanup of the styling of the edit grid Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=73c6f1ca3bb28fe33ac9d5eac8154a46f331fc58 Author: Surinder Kumar Modified Files -- web/pgadmin/tools/sqleditor/static/css/sqleditor.css

Re: [pgadmin-hackers] [pgAdmin4][Patch]: UI improvements in data-grid/query tool

2017-06-08 Thread Dave Page
nged to #888 to match with bottom > border > (#888) > , now the header can easily be distinguished from the grid rows border > styling. > > Please review and let me know if any changes. > > > On Thu, Jun 8, 2017 at 7:51 PM, Dave Page wrote: >> >> Exact

Re: [pgadmin-hackers] Style guide live website - looking for feedback

2017-06-08 Thread Dave Page
where these fonts are used now, and then we can > think about if that's how we actually want to use them. > >> >> - You've obviously created classes for these styles (e.g. text-14). >> Let's make sure they are all actually defined in the stylesheets for >>

Re: [pgadmin-hackers] [pgAdmin4][Patch][RM_2450] : Internal Server Error displayed if click on statistic tab for PG 10 server

2017-06-08 Thread Dave Page
, I think the directory name should be 10_plus shouldn't it? Will that work with the template search code? If not, can you please fix that as well? Thanks! -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Compa

Re: [pgadmin-hackers] [pgAdmin4][Patch]: UI improvements in data-grid/query tool

2017-06-08 Thread Dave Page
; data-grid right side seems to fixed. >> >> PFA patch and review. >> >> >> Thanks, >> Surinder >> >> >> -- >> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) >> To make changes to your subscription: >&g

Re: [pgadmin-hackers] [pgAdmin4][PATCH] Improvements to Query Results Grid User Experience

2017-06-08 Thread Dave Page
Hi On Thu, Jun 8, 2017 at 2:18 PM, Sarah McAlear wrote: > On Thu, Jun 8, 2017 at 8:34 AM, Dave Page wrote: >> - There are changes to SlickGrid, the addition of a function to scroll >> a column into view. Is this submitted upstream and accepted? > > We have submitted the c

Re: [pgadmin-hackers] Fix for RM2421 [pgAdmin4][patch]

2017-06-08 Thread Dave Page
Thanks, applied. On Thu, Jun 8, 2017 at 1:59 PM, Harshal Dhumal wrote: > Please find attached rebased patch. > > -- > Harshal Dhumal > Sr. Software Engineer > > EnterpriseDB India: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > On Wed, Jun

[pgadmin-hackers] pgAdmin 4 commit: Fix integer/numeric validation on various dialogues.

2017-06-08 Thread Dave Page
Fix integer/numeric validation on various dialogues. Fixes #2421 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=aa400cbc12f34f219322a2b62f9f51e21d031f0e Author: Harshal Dhumal Modified Files -- .../templates/foreign_tables/js/for

Re: [pgadmin-hackers] [pgAdmin4][PATCH] Fix handling of double precision[] type

2017-06-08 Thread Dave Page
ackers@postgresql.org) > To make changes to your subscription: > http://www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers

[pgadmin-hackers] pgAdmin 4 commit: Fix the handling of double precision[] type. Fixes #

2017-06-08 Thread Dave Page
Fix the handling of double precision[] type. Fixes #1928 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=3bcbc505258e4583d94a40c6e0506046bdc21b98 Author: Murtuza Zabuawala Modified Files -- web/pgadmin/utils/driver/psycopg2/__ini

Re: [pgadmin-hackers] [pgAdmin4][Patch][Feature_1535]: Pressing ESC from within a dialog box should act like "Cancel" button

2017-06-08 Thread Dave Page
r behaviour in some other apps. Let's leave it as you've done it :-) Please add support for the properties dialogues, then I think we're good to go. Thanks! -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterpris

Re: [pgadmin-hackers] [pgAdmin4][Patch][Feature_1533]: Set focus on first field when new dialog window is presented

2017-06-08 Thread Dave Page
/www.postgresql.org/mailpref/pgadmin-hackers > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your

[pgadmin-hackers] pgAdmin 4 commit: Set focus on the first enabled field when a dialogue

2017-06-08 Thread Dave Page
Set focus on the first enabled field when a dialogue is opened. Fixes #1533 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=c9b717ed68797740a6ea8d3480627fe21777a8a8 Author: Surinder Kumar Modified Files -- web/pgadmin/browser/temp

Re: [pgadmin-hackers] [pgAdmin4] [PATCH] History Tab rewrite in React

2017-06-08 Thread Dave Page
here repo. Once I can apply a version of this, how should I be running the linter and the unit tests? -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgadmin-hackers mailing

  1   2   3   4   5   6   7   8   9   10   >