Re: CSS SonarQube fixes - Mailing list pgadmin-hackers
From | Pramod Ahire |
---|---|
Subject | Re: CSS SonarQube fixes |
Date | |
Msg-id | 63DC2029-E5A4-4A6A-83D8-B682E425FC10@enterprisedb.com Whole thread Raw |
In response to | Re: CSS SonarQube fixes (Akshay Joshi <akshay.joshi@enterprisedb.com>) |
List | pgadmin-hackers |
Hi Akshay,
Created new patch as you suggested by cloning new repo & making changes in it.
Now the patch is completely based on pgadmin repo. Attached for your reference.
Please feel free to write if anything missed or not working.
Thanks!
From: Pramod Ahire <pramod.ahire@enterprisedb.com>
Date: Monday, 6 July 2020 at 9:05 AM
To: Akshay Joshi <akshay.joshi@enterprisedb.com>
Cc: Nagesh Dhope <nagesh.dhope@enterprisedb.com>, Postgres Enterprise Manager <pem@enterprisedb.com>, pgadmin-hackers <pgadmin-hackers@postgresql.org>
Subject: Re: CSS SonarQube fixes
Hi Akshay,
Sorry for the inconvenience, will do as you suggested thanks !
From: Akshay Joshi <akshay.joshi@enterprisedb.com>
Date: Friday, 3 July 2020 at 6:46 PM
To: Pramod Ahire <pramod.ahire@enterprisedb.com>
Cc: Nagesh Dhope <nagesh.dhope@enterprisedb.com>, Postgres Enterprise Manager <pem@enterprisedb.com>, pgadmin-hackers <pgadmin-hackers@postgresql.org>
Subject: Re: CSS SonarQube fixes
Hi Pramod
The patch is not applied, take the latest code of pgAdmin, and then create the patch.
On Fri, Jul 3, 2020 at 5:33 PM Pramod Ahire <pramod.ahire@enterprisedb.com> wrote:
Hi Akshay,
Attached updated patch, can you try & let me know if it works ?
@Nagesh Please commit the patch for PEM earlier sent over email, as wizard css changes are not for pgadmin excluded it in latest patch.
Thanks & sorry for the inconvenience.
From: Akshay Joshi <akshay.joshi@enterprisedb.com>
Date: Friday, 3 July 2020 at 3:34 PM
To: Pramod Ahire <pramod.ahire@enterprisedb.com>
Cc: Nagesh Dhope <nagesh.dhope@enterprisedb.com>, Postgres Enterprise Manager <pem@enterprisedb.com>, pgadmin-hackers <pgadmin-hackers@postgresql.org>
Subject: Re: CSS SonarQube fixes
Not able to apply the patch.
On Fri, Jul 3, 2020 at 12:15 PM Pramod Ahire <pramod.ahire@enterprisedb.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.
From: Nagesh Dhope <nagesh.dhope@enterprisedb.com>
Date: Wednesday, 1 July 2020 at 10:57 AM
To: Pramod Ahire <pramod.ahire@enterprisedb.com>
Cc: Postgres Enterprise Manager <pem@enterprisedb.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@postgresql.org.
On Tue, Jun 30, 2020 at 4:44 PM Nagesh Dhope <nagesh.dhope@enterprisedb.com> wrote:
Hi Team,
I am reviewing this patch.
On Mon, Jun 29, 2020 at 4:54 PM Pramod Ahire <pramod.ahire@enterprisedb.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
C: +91-020-66449600/601
D: +91-9028697679
--
Thanks,
Nagesh
--
Thanks,
Nagesh
--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246
--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246
Attachment
pgadmin-hackers by date: