Re: pgAdmin4 PATCH: Domain Module
От | Dave Page |
---|---|
Тема | Re: pgAdmin4 PATCH: Domain Module |
Дата | |
Msg-id | CA+OCxoxrV-_=UWOZ2WnJWA9z2E_hRbhYxpJYFNQyMAFDwsjVkw@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: pgAdmin4 PATCH: Domain Module (Khushboo Vashi <khushboo.vashi@enterprisedb.com>) |
Ответы |
Re: pgAdmin4 PATCH: Domain Module
(Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
|
Список | pgadmin-hackers |
Hi Almost there :-s - The hint for default should be a placeholder in the textbox itself (like combos have "Select from the list" - I should be able to check the "Validate?" option on one or more constraints from within the Domain dialogue - Please ensure the capitalisation of all property labels is consistent - it should be "Base type" not "Base Type", "System domain?" not "System Domain?" etc. - The check constraint reverse engineered SQL should include the path to the constraint, e.g. -- CHECK: schema.domain.check_at Once that's done, it can be committed I think. Thanks. On Wed, Mar 23, 2016 at 7:27 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: > Updated one comment. > > On Wed, Mar 23, 2016 at 12:48 PM, Khushboo Vashi > <khushboo.vashi@enterprisedb.com> wrote: >> >> Hi, >> >> Please find attached updated patch for the Domains module. >> >> On Wed, Mar 16, 2016 at 9:40 PM, Dave Page <dpage@pgadmin.org> wrote: >>> >>> Hi >>> >>> On Wed, Mar 16, 2016 at 2:03 PM, Khushboo Vashi >>> <khushboo.vashi@enterprisedb.com> wrote: >>> > Hi, >>> > >>> > Please find the updated Domain Module Patch. >>> > >>> > To test this patch, please apply Backgrid Textarea Cell Patch before >>> > this. >>> >>> Thanks. I believe with the following fixes, we'll be done :-) >>> >>> - Default values should be auto-quoted when necessary (ie. strings, on >>> a text-based domain). >> >> As per our discussion, this should leave unquoted. > > And also added a hint to the field stated 'Enter an expression or a value.' >> >> >>> - "System Domain?" should be in the General section, between owner and >>> comment. >> >> Done >>> >>> - The switches should use the same colouring/styling as other objects, >>> e.g. >>> >>> options: { >>> 'onText': 'Yes', 'offText': 'No', >>> 'onColor': 'success', 'offColor': 'primary', >>> 'size': 'small' >>> } >> >> Done >>> >>> - Please remove the Schema property from the main properties tab (not >>> the properties dialogue). >> >> Done >>> >>> - No icon is show for Checks on the Dependents tab for a domain. >> >> Done >>> >>> - The add button on the Security Labels tab is spelt "Add". Why is >>> that? Other instances of this grid use "ADD" which is the default in >>> backform.pgadmin.js. >> >> Done >>> >>> - Dependencies on domain check constraints are listed as being on a >>> "Type" not a "Domain". >> >> Done >>> >>> - If adding a domain constraint using the grid on the Domain dialogue, >>> I cannot specify "NOT VALID". We need a checkbox for that in a narrow >>> columns at the end. Unchecking it for an existing constraint should be >>> the equivalent of doing "ALTER DOMAIN ... VALIDATE CONSTRAINT" >> >> Done >>> >>> - If I switch the "Don't Validate" switch on a constraint, there are >>> leading blank lines in the generated SQL. The same occurs when adding >>> a comment to a constraint. >> >> Done >>> >>> - I think we need to reverse the meaning of "Don't Validate" and >>> rename to match the "Valid?" field that's on the properties list. >>> Otherwise it's not clear they're the same thing. >> >> Done >>> >>> - s/Not Null/Not Null?/ >> >> Done >>> >>> >>> >>> -- >>> Dave Page >>> Blog: http://pgsnake.blogspot.com >>> Twitter: @pgsnake >>> >>> EnterpriseDB UK: http://www.enterprisedb.com >>> The Enterprise PostgreSQL Company >> >> > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company
В списке pgadmin-hackers по дате отправления: