Re: Bug #3083 fix

Поиск
Список
Период
Сортировка
От Joao De Almeida Pereira
Тема Re: Bug #3083 fix
Дата
Msg-id CAE+jjamnK3vVZYqs_6+4yVxUPgpMbgM9NM06m1L4ytaMbkXpVQ@mail.gmail.com
обсуждение исходный текст
Ответ на Bug #3083 fix  (Neethu Mariya Joy <neethumariyajoy@gmail.com>)
Ответы Re: Bug #3083 fix  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Список pgadmin-hackers
Hello Neethu,
We passed the patch through our CI pipeline and all tests pass.
The code looks good, but we are trying to decouple files as much  as we can so that we do not end up with files with over 1000 lines, that are hard to read and to maintain. Also we are trying to create Unit Tests to have more coverage in our Javascript code.

Can you split the new implementation code into it's own file and create some tests to ensure the behavior will not be broken in the future?iYou have some examples on: pgadmin/browser/server_groups/servers/databases/external_tables/*

Thanks
Joao

On Thu, Mar 1, 2018 at 10:37 AM Neethu Mariya Joy <neethumariyajoy@gmail.com> wrote:
Hi,
I am Neethu Mariya Joy, an undergraduate pursuing BE in Computer Science at BITS Pilani.

I've attempted to fix https://redmine.postgresql.org/issues/3083. Since the textarea resize feature is the default HTML feature, I have not changed it. Instead, I've added draggable borders to the wrapper which expands the textarea inside it.

I'm attaching my patch as bug3083.diff below as per the contribution guidelines.

Hope this helps. Thank you for your consideration!

Sincerely,
Neethu Mariya Joy


В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Joao De Almeida Pereira
Дата:
Сообщение: Re: Bug #2309 fix
Следующее
От: Joao De Almeida Pereira
Дата:
Сообщение: Re: [pgAdmin4][RM#3129] handle encoding issue in File manager