Re: [pgAdmin][RM-6687]: [React] Port Grant Wizard to react
От | Nikhil Mohite |
---|---|
Тема | Re: [pgAdmin][RM-6687]: [React] Port Grant Wizard to react |
Дата | |
Msg-id | CAOBg0AP+NYHkesPvAK9wCaLJd9HZbDRqc=k9Bumm1P_+K5zyWg@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: [pgAdmin][RM-6687]: [React] Port Grant Wizard to react (Akshay Joshi <akshay.joshi@enterprisedb.com>) |
Ответы |
Re: [pgAdmin][RM-6687]: [React] Port Grant Wizard to react
|
Список | pgadmin-hackers |
Hi Akshay,
Please find the updated patch (v3) resolved all review comments.
On Tue, Sep 14, 2021 at 6:13 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi NikhilFollowing are the GUI related review comments:
- Remove the header string "Object selection (step 1 of 3) as we are already showing steps on the left side of the wizard.
Done
- Rename step 1 to "Object Selection" step 2 to "Privilege Selection" and step 3 to "Review Selection".
Done
- The search button is not implemented on the first page (object selection).
Done
- The help button is not implemented at the bottom left corner of the wizard.
Done
- Duplicate entries are found on the object selection page. (Seems like an old issue without this patch)
I tried but was unable to detect the exact issue.
- Checkbox checked color should be matched with the theme.
Done
- Scrollbar alignment is not correct and spaces at the right side and bottom look ugly.
Done
- The grantor should be set and read-only.
Done
- Showing all privileges instead of based on the selected node. Please refer to existing behavior.
Done
Refer screenshot with all above review comments:On Tue, Sep 14, 2021 at 1:19 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Hackers,Please find the updated patch for grant wizard porting to react. (Please ignore the previous patch)On Tue, Sep 14, 2021 at 10:05 AM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Regards,Nikhil Mohite--Thanks & RegardsAkshay JoshipgAdmin Hacker | Principal Software ArchitectEDB PostgresMobile: +91 976-788-8246
Regards,
Nikhil Mohite
Вложения
В списке pgadmin-hackers по дате отправления: