Обсуждение: [pgAdmin4][Patch]: Minor JS Fix

Поиск
Список
Период
Сортировка

[pgAdmin4][Patch]: Minor JS Fix

От
Khushboo Vashi
Дата:
Hi,

Please find the attached monir patch to fix the JS issue.

Thanks,
Khushboo
Вложения

Re: [pgAdmin4][Patch]: Minor JS Fix

От
Joao De Almeida Pereira
Дата:
Hi,
The code looks fine, all tests pass and linter.

Once again, not to sound like a broken record, we could have done some much more to improve our code base....
Why didn't we extract that function into a more testable place?

In order for the code to become more clean and modular, we should try to always write tests whenever we make changes to the code, even if that means extracting a small portion (ie. the onLoad function)

Thanks
Victoria & Joao

On Tue, May 15, 2018 at 12:44 AM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached monir patch to fix the JS issue.

Thanks,
Khushboo