Re: PATCH: Schema/Catalog Node [pgAdmin4]

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: PATCH: Schema/Catalog Node [pgAdmin4]
Дата
Msg-id CA+OCxowNOWJPQdh08nZLZHzAXmP+g_xXfOvEzaVLT5dciJ3=5g@mail.gmail.com
обсуждение исходный текст
Ответ на Re: PATCH: Schema/Catalog Node [pgAdmin4]  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Ответы Re: PATCH: Schema/Catalog Node [pgAdmin4]  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Список pgadmin-hackers
Hi

On Sun, Feb 28, 2016 at 6:49 AM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Hi Dave,

As discussed, I have worked on this patch to improve some code level changes.
(because - Murtuza was not available due to some other engagement.)

Can you please review it, and share your feedback?

I think it's  mostly there. I've attached an updated patch where I've fixed a few minor issues as I read through the code. The following (likely simple) issues need to be fixed:

- Dropping a schema fails with an error message of "schema/pg/9.2_plus/sql/get_name.sql". 

- Creating a schema appears to fail with "'data' is undefined", however the schema is created, it's just that the dialogue doesn't close and the new schema isn't added to the tree.

- There is some discrepancy between default privileges as displayed on the properties summary, the edit dialogue, and the RE-SQL. As you can see in the screenshot, the SQL just GRANTS ALL, and the properties panel doesn't show anything.

Thanks.

-- 
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Вложения

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

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: [pgAdmin4][Patch]: Macros for the functions module
Следующее
От: Dave Page
Дата:
Сообщение: pgAdmin 4 commit: Security Label and ACL template macros: