[pgadmin-hackers] Re: [pgAdmin4][Patch][RM2399]: Disabled row's background colordisappeared on save in View data - Mailing list pgadmin-hackers

From Surinder Kumar
Subject [pgadmin-hackers] Re: [pgAdmin4][Patch][RM2399]: Disabled row's background colordisappeared on save in View data
Date
Msg-id CAM5-9D_r6LF2get9CLxbUvQULFRM8sf7PahMCbFrX4sfVLxo+g@mail.gmail.com
Whole thread Raw
In response to [pgadmin-hackers] [pgAdmin4][Patch][RM2399]: Disabled row's background colordisappeared on save in View data  (Surinder Kumar <surinder.kumar@enterprisedb.com>)
Responses Re: [pgadmin-hackers] Re: [pgAdmin4][Patch][RM2399]: Disabled row'sbackground color disappeared on save in View data
List pgadmin-hackers
Hi

The regression test cases for 'copy row' was failing.
This patch was assuming that each selected row will have primary key due to selection was not working. Fixed.

Please find revised patch.

Thanks
Surinder Kumar

On Mon, May 15, 2017 at 1:13 PM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
Hi,

This patch contains following changes:

1) When a new row is added, allow to delete if changes are not saved.

2) Disable new row selection if primary key is not given but changes are saved on server.

3) Copy and paste one or more rows, then add new row doesn't work(entered value doesn't appear).

4) After deleting a row, add new row doesn't work.

5) New row added with explicitly given primary key should not disabled.


Please review.

Thanks,
Surinder Kumar

Attachment

pgadmin-hackers by date:

Previous
From: Neel Patel
Date:
Subject: [pgadmin-hackers] [pgAdmin4][runtime][patch]: Fix compilation error for QtWebEngine
Next
From: Dave Page
Date:
Subject: [pgadmin-hackers] pgAdmin 4 commit: While building runtime with QtWebEngine,we found 2 c