Re: CSS SonarQube fixes

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Pramod Ahire <pramod(dot)ahire(at)enterprisedb(dot)com>
Cc: Nagesh Dhope <nagesh(dot)dhope(at)enterprisedb(dot)com>, Postgres Enterprise Manager <pem(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: CSS SonarQube fixes
Date: 2020-07-03 10:03:49
Message-ID: CANxoLDfjZurx-4K0=_4Ggp39_VZr-MreCt3=j8UOdtjL7RJxmA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Not able to apply the patch.

On Fri, Jul 3, 2020 at 12:15 PM Pramod Ahire <pramod(dot)ahire(at)enterprisedb(dot)com>
wrote:

> Hi Nagesh,
>
>
>
> Done the changes as you suggested, please find attached patch for same.
>
>
>
> Also coping pgadmin-hackers for integration.
>
>
>
> Thanks & feel free to let me know, if missed anything.
>
>
>
> *Pramod Ahire*
>
> *Software Engineer*
>
>
>
> [image:
> https://lh4.googleusercontent.com/U1erEuyI_d0xEUA0CrKjwx9aWQ52HNCxc38dMsLP-ZrLgfVNrhsrNobxlmeOdb1kMPtrrxcUwEHZgbGJC4R0qR4r1sBZa_z9R8ihFRaP2Hr_Wnhq6HcIQHe1ZoviDPwUkTdzNcg]
>
>
>
> C: +91-020-66449600/601
>
> D: +91-9028697679
>
> *edbpostgres.com <http://edbpostgres.com>*
>
>
>
>
>
> *From: *Nagesh Dhope <nagesh(dot)dhope(at)enterprisedb(dot)com>
> *Date: *Wednesday, 1 July 2020 at 10:57 AM
> *To: *Pramod Ahire <pramod(dot)ahire(at)enterprisedb(dot)com>
> *Cc: *Postgres Enterprise Manager <pem(at)enterprisedb(dot)com>
> *Subject: *Re: CSS SonarQube fixes
>
>
>
> Hi Pramod,
>
>
>
> Remove commented styles from below files.
>
> web/pgadmin/misc/file_manager/static/css/file_manager.css
> web/pgadmin/misc/file_manager/static/scss/_file_manager.scss
> web/pgadmin/misc/static/explain/scss/_explain.scss
>
>
>
> Apart from this, the patch looks good. Please send the updated patch to
> pgadmin-hackers(at)postgresql(dot)org(dot)
>
>
>
>
>
> On Tue, Jun 30, 2020 at 4:44 PM Nagesh Dhope <
> nagesh(dot)dhope(at)enterprisedb(dot)com> wrote:
>
> Hi Team,
>
> I am reviewing this patch.
>
>
>
> On Mon, Jun 29, 2020 at 4:54 PM Pramod Ahire <
> pramod(dot)ahire(at)enterprisedb(dot)com> wrote:
>
> Hi Team,
>
>
>
> Please find few CSS fixes for SonarQube.
>
> It consists below types of issues:
>
>
>
> - Duplicate CSS removal
> - Few standard syntax applied on classes
> - Double semicolon removal
> - Other minor
>
>
>
> Had tested at my end, & verified. Please do let me know if missed anything
> thanks !
>
>
>
> *Pramod Ahire*
>
> *Software Engineer*
>
>
>
> [image:
> https://lh4.googleusercontent.com/U1erEuyI_d0xEUA0CrKjwx9aWQ52HNCxc38dMsLP-ZrLgfVNrhsrNobxlmeOdb1kMPtrrxcUwEHZgbGJC4R0qR4r1sBZa_z9R8ihFRaP2Hr_Wnhq6HcIQHe1ZoviDPwUkTdzNcg]
>
>
>
> C: +91-020-66449600/601
>
> D: +91-9028697679
>
> *edbpostgres.com <http://edbpostgres.com>*
>
>
>
>
>
>
> --
>
> Thanks,
>
> Nagesh
>
>
>
>
> --
>
> Thanks,
>
> Nagesh
>

--
*Thanks & Regards*
*Akshay Joshi*

*Sr. Software Architect*
*EnterpriseDB Software India Private Limited*
*Mobile: +91 976-788-8246*

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Satish V 2020-07-03 11:00:32 Reg - [RM-3814] Backup dialog does not show details of error if file name box kept empty
Previous Message Akshay Joshi 2020-07-03 09:10:43 Re: [pgAdmin4][Patch] - SonarQube Issues - 8 (Variables should not be shadowed)