| From: | Yosry Muhammad <yosrym93(at)gmail(dot)com> |
|---|---|
| To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
| Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
| Subject: | Re: Query Tool JS Minimal Refactoring |
| Date: | 2019-08-19 07:54:54 |
| Message-ID: | CAFSMqn_44ToOJtkV97JcGi83K78yfdOr7Hp8QxJC7v7N-5CMig@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgadmin-hackers |
Jasmine tests passed on my machine, I will take another look once I have
access to my machine.
On Mon, Aug 19, 2019, 7:57 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Yosry
>
> Jasmine tests are failing, can you please fix those and resend the patch.
>
> On Fri, Aug 16, 2019 at 11:23 PM Yosry Muhammad <yosrym93(at)gmail(dot)com>
> wrote:
>
>> Hi hackers,
>>
>> Please find attached a patch with minimal refactoring of:
>> web/pgadmin/tools/sqleditor/static/js/sqleditor.js
>>
>> This includes merging 2 redundant functions into one and renaming some
>> functions to have more expressive and consistent names.
>>
>> Please review !
>> Thanks.
>> --
>> *Yosry Muhammad Yosry*
>>
>> Computer Engineering student,
>> The Faculty of Engineering,
>> Cairo University (2021).
>> Class representative of CMP 2021.
>> https://www.linkedin.com/in/yosrym93/
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Akshay Joshi | 2019-08-19 12:08:06 | Re: Transaction Status Python Tests |
| Previous Message | Yosry Muhammad | 2019-08-19 07:52:23 | Re: Transaction Status Python Tests |