Обсуждение: [pgAdmin4][RM#3105] Allow user to insert new row if PK is in capital letters
[pgAdmin4][RM#3105] Allow user to insert new row if PK is in capital letters
От
Murtuza Zabuawala
Дата:
Hi,
PFA patch to fix the issue user was not able to insert new row in the table via 'view data' if the PK is in capital letters.
--
Regards,
Вложения
Re: [pgAdmin4][RM#3105] Allow user to insert new row if PK is incapital letters
От
Joao De Almeida Pereira
Дата:
Hello Murtuza,
Just reviewed your patch, and it looks good.
Do you think you can wrap some tests around this behavior to ensure it doesn't happen again in the future?
Thanks
Joao
On Thu, Feb 15, 2018 at 8:34 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,PFA patch to fix the issue user was not able to insert new row in the table via 'view data' if the PK is in capital letters.--Regards,
Re: [pgAdmin4][RM#3105] Allow user to insert new row if PK is incapital letters
От
Murtuza Zabuawala
Дата:
Sure, will do that.
--
Regards,
On Thu, Feb 15, 2018 at 7:34 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:
Hello Murtuza,Just reviewed your patch, and it looks good.Do you think you can wrap some tests around this behavior to ensure it doesn't happen again in the future?ThanksJoaoOn Thu, Feb 15, 2018 at 8:34 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi,PFA patch to fix the issue user was not able to insert new row in the table via 'view data' if the PK is in capital letters.--Regards,
Re: [pgAdmin4][RM#3105] Allow user to insert new row if PK is incapital letters
От
Murtuza Zabuawala
Дата:
Hi,
PFA updated patch, which includes feature test to test the scenario going forward.
--
Regards,
On Thu, Feb 15, 2018 at 7:53 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Sure, will do that.--Regards,On Thu, Feb 15, 2018 at 7:34 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:Hello Murtuza,Just reviewed your patch, and it looks good.Do you think you can wrap some tests around this behavior to ensure it doesn't happen again in the future?ThanksJoaoOn Thu, Feb 15, 2018 at 8:34 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi,PFA patch to fix the issue user was not able to insert new row in the table via 'view data' if the PK is in capital letters.--Regards,
Вложения
Re: [pgAdmin4][RM#3105] Allow user to insert new row if PK is incapital letters
От
Joao De Almeida Pereira
Дата:
Hello Murtuza,
I believe it is a bit too much to have a feature test for this. A unit test around the function in question could have sufficed, specially because we are not yet running them in Jenkins, but I think it is ok.
So I review the patch, applied and run our CI and everything is green. So I believe this patch is good to be merged.
Thanks
Joao
On Fri, Feb 16, 2018 at 5:28 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,PFA updated patch, which includes feature test to test the scenario going forward.--Regards,On Thu, Feb 15, 2018 at 7:53 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:Sure, will do that.--Regards,On Thu, Feb 15, 2018 at 7:34 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:Hello Murtuza,Just reviewed your patch, and it looks good.Do you think you can wrap some tests around this behavior to ensure it doesn't happen again in the future?ThanksJoaoOn Thu, Feb 15, 2018 at 8:34 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:Hi,PFA patch to fix the issue user was not able to insert new row in the table via 'view data' if the PK is in capital letters.--Regards,
Thanks guys, patch applied.
On Fri, Feb 16, 2018 at 2:30 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:
Hello Murtuza,I believe it is a bit too much to have a feature test for this. A unit test around the function in question could have sufficed, specially because we are not yet running them in Jenkins, but I think it is ok.So I review the patch, applied and run our CI and everything is green. So I believe this patch is good to be merged.ThanksJoaoOn Fri, Feb 16, 2018 at 5:28 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi,PFA updated patch, which includes feature test to test the scenario going forward.--Regards,On Thu, Feb 15, 2018 at 7:53 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Sure, will do that.--Regards,On Thu, Feb 15, 2018 at 7:34 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:Hello Murtuza,Just reviewed your patch, and it looks good.Do you think you can wrap some tests around this behavior to ensure it doesn't happen again in the future?ThanksJoaoOn Thu, Feb 15, 2018 at 8:34 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi,PFA patch to fix the issue user was not able to insert new row in the table via 'view data' if the PK is in capital letters.--Regards,
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company