Jenkins build is back to normal : pgadmin4-master-python33 #212

2017-07-03 Thread pgAdmin 4 Jenkins
See

pgAdmin 4 commit: Remove unnecessary whitespace from stored procedure S

2017-07-03 Thread Dave Page
Remove unnecessary whitespace from stored procedure SQL. Fixes #2146 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=0aa763b70680707136036ed1b7d3f994e70161de Author: Murtuza Zabuawala Modified Files

pgAdmin 4 commit: Allow debugging of functions in packages. Fixes #1948

2017-07-03 Thread Dave Page
Allow debugging of functions in packages. Fixes #1948 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=5c7dcaba81c55846f5d9a85f1226abdee8cc1856 Author: Murtuza Zabuawala Modified Files --

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

2017-07-03 Thread Surinder Kumar
Hi All, *​Things to discuss:* How to differentiate between a static and template JS ​​ . ​In Webpack we have to resolve paths to the modules same as in requirejs.config(base.html). but previously we were using 'current_app.javascripts' which holds the the path references for modules registered.

pgAdmin 4 commit: Fix deletion of table rows with the column definition

2017-07-03 Thread Dave Page
Fix deletion of table rows with the column definition having NOT NULL TRUE and HAS NO DEFAULT VALUE. Fixes #2527 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=f6b879233c1c9f554a43094e1d82d3e16cff23fa Author: Khushboo Vashi

Jenkins build is back to normal : pgadmin4-master-python34 #201

2017-07-03 Thread pgAdmin 4 Jenkins
See

Re: Tree view icon samples

2017-07-03 Thread Dave Page
Hi On Mon, Jul 3, 2017 at 10:25 AM, Chethana Kumar < chethana.ku...@enterprisedb.com> wrote: > Hi Dave, > > I have updated the icons "Conversions, Function and Function Trigger" as > per you suggestions. > Nice. > > I feel the curved shapes won't look clear in 16px measurement which is >

Re: [pgAdmin4][Patch]: Fixed the issue related to Domain Constraint module

2017-07-03 Thread Dave Page
Hi On Mon, Jul 3, 2017 at 6:10 AM, Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > > > On Mon, Jul 3, 2017 at 1:10 PM, Khushboo Vashi < > khushboo.va...@enterprisedb.com> wrote: > >> Hi, >> >> The Domain Constraint module doesn't load as it can not find the >> domain_constraints.js

Re: [pgAdmin4][Patch] To fix the issues in Debugger module

2017-07-03 Thread Dave Page
Hi On Fri, Jun 30, 2017 at 10:15 AM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Hi, > > Please find patch to fix the below issues, > > 1) Debugger page layout > RM#2512 > > 2) To fix the issue when you user clicks on Stop button and gets > connection lost error. > RM#2511 >

pgAdmin 4 commit: Improve the debugger's default page layout, and preve

2017-07-03 Thread Dave Page
Improve the debugger's default page layout, and prevent attempts to continue debugging after the user has stopped. Fixes #2512, Fixes #2511 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=2d5e55964ad811690ac1b52efdc632e8b485e943 Author:

Re: [pgadmin-hackers] [Design update] Style guide for pgAdmin4

2017-07-03 Thread Robert Eckhardt
On Mon, Jul 3, 2017 at 5:36 AM, Dave Page wrote: > > > On Fri, Jun 30, 2017 at 12:19 PM, Shirley Wang wrote: > >> Hello! >> >> Currently the app uses 'monospace' to define the font family for SQL >> queries and messages, which looks a little squished when in

Re: Tree view icon samples

2017-07-03 Thread Dave Page
Hi On Mon, Jul 3, 2017 at 8:53 AM, Chethana Kumar < chethana.ku...@enterprisedb.com> wrote: > Hi Dave, > > I am attaching the updated icons along with new icons added. > > Also I have tried to address the issues raised in our previous review - > > 1. Cast and Conversion: Eliminated the texty

[pgAdmin4][Patch] To fix the issue in Debugger & Procedure module for EPAS server

2017-07-03 Thread Murtuza Zabuawala
Hi, PFA patch to fix the issue where 'Debug' option was not displaying on Package -> Function if the user if non-super user. RM#1948 Another minor issue where there was a whitespace on SQL panel while displaying reversed engineering sql for Procedure. RM#2146 -- Regards, Murtuza Zabuawala

pgAdmin 4 commit: Fix clipboard handling with large datasets. Fixes #24

2017-07-03 Thread Dave Page
Fix clipboard handling with large datasets. Fixes #2489 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=1977a5fcda44b78b00d6eaac2e6a99df355d5105 Author: Harshal Dhumal Modified Files --

Re: [pgAdmin4][Patch]: Fixed the issue related to Domain Constraint module

2017-07-03 Thread Khushboo Vashi
On Mon, Jul 3, 2017 at 1:10 PM, Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi, > > The Domain Constraint module doesn't load as it can not find the > domain_constraints.js file. > To fix this issue, renamed the domain_constraint.js to > domain_constraints.js. > > RM #2529

Re: [pgAdmin4][Patch]: Fixed couple of minor issues

2017-07-03 Thread Khushboo Vashi
On Mon, Jul 3, 2017 at 11:57 AM, Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi, > > 1. When we try to delete the table row with the column definition having > NOT NULL TRUE and HAS NO DEFAULT VALUE then it fails. > Please find the attached patch for the same.

pgAdmin 4 commit: Fix a syntax error in the SQL Editor.

2017-07-03 Thread Dave Page
Fix a syntax error in the SQL Editor. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=4e21cdb64bc68c20d195a63c17b2f926e75a Author: Harshal Dhumal Modified Files --

Re: Patch for sqleditor js syntax issue

2017-07-03 Thread Dave Page
Thanks, patch applied. On Fri, Jun 30, 2017 at 3:27 PM, Harshal Dhumal < harshal.dhu...@enterprisedb.com> wrote: > Hi, > > Please find attached minor patch which fixes js syntax issue in > sqleditor.js > > > -- > *Harshal Dhumal* > *Sr. Software Engineer* > > EnterpriseDB India:

Re: [pgAdmin4][Patch] To fix the duplicate entry in History panel

2017-07-03 Thread Dave Page
Thanks, patch applied. On Fri, Jun 30, 2017 at 2:26 PM, George Gelashvili wrote: > Nice! Looks good. We were investigating this bug before you sent this and > it looks like your patch resolves the duplicate calls to > update_msg_history > > Cheers, > Sarah and George > >

pgAdmin 4 commit: Fix an issue where duplicate entry is made of success

2017-07-03 Thread Dave Page
Fix an issue where duplicate entry is made of successful executed query, one with status message and one without status message. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=1e427890de93f83d2b1ffb0e25b1c6f6d8bd8389 Author: Murtuza

Re: [pgAdmin4][Patch]: Fixed couple of minor issues

2017-07-03 Thread Dave Page
Are there RMs for these? On Mon, Jul 3, 2017 at 2:27 AM, Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi, > > 1. When we try to delete the table row with the column definition having > NOT NULL TRUE and HAS NO DEFAULT VALUE then it fails. > Please find the attached patch for

Re: [pgadmin-hackers] [Design update] Style guide for pgAdmin4

2017-07-03 Thread Dave Page
On Fri, Jun 30, 2017 at 12:19 PM, Shirley Wang wrote: > Hello! > > Currently the app uses 'monospace' to define the font family for SQL > queries and messages, which looks a little squished when in uppercase. PT > mono is another monospaced font (available on Google fonts) that

Re: Tree view icon samples

2017-07-03 Thread Dave Page
Hi On Friday, June 30, 2017, Strauch, Sheldon wrote: > Dave et al, > > Might I suggest flipping the foreign key icon left for right? That way, > the key "points" to the left, back toward something pre-existing whilst the > primary key "points" to the right, forward towards

[pgAdmin4][Patch]: Fixed the issue related to Domain Constraint module

2017-07-03 Thread Khushboo Vashi
Hi, The Domain Constraint module doesn't load as it can not find the domain_constraints.js file. To fix this issue, renamed the domain_constraint.js to domain_constraints.js. Please find the attached fix. diff --git