Thread: [pgAdmin][RM-6133]: [React] Port schema diff to React
Attachment
// it('SchemaDiff Module', () => { + // // let baseUrl = url_for('schema_diff.initialize'); + // // networkMock.onGet(url_for('debugger.start_listener', {'trans_id': params.transId})).reply(200, {'success':1,'errormsg':'','info':'','result':null,'data':{'status':true,'result':2}}); + // // console.log('URL:: ', url_for('schema_diff.initialize', {})); + // networkMock.onGet(url_for('schema_diff.initialize', {})).reply(200, + // { + // 'success':1,
2. Do not use jquery to bind events.
+ if(window.opener) {
+ $(window).on('unload', function(ev) {
3. Functions like these which do not use state setters can be moved outside the component for better readability/performance.
+ function generateFinalScript(script_array, scriptHeader, script_body) {
+ _.each(Object.keys(script_array).reverse(), function (s) {
+ if (script_array[s].length > 0) {
4. Use gettext
+export const FILTER_NAME = {
+ IDENTICAL : 'Identical',
+ DIFFERENT :'Different',
5. This is not used anywhere. You can remove it:
def script():
"""render the required javascript"""
return Response(
- response=render_template("schema_diff/js/schema_diff.js", _=gettext),
+ response=render_templ
Hi Hackers,Please find attached the patch for RM-6133 : [React] Port schema diff to React.This patch also included the changes for removing SlickGrid from pgAdmin.--
- Reduce some space from source selection and target selection.
- Inside the select control, make the server group label bold.
- There is currently a search icon instead of a filter icon. On the button, change the cursor pointer to none.
- Add a help button near the 'Filter' button.
- The spinner and comparison status string should be bolded or larger.
- The distance between the 'Generate Script' and 'Filter' buttons should be the same, it should not change on resizing of the browser. or open schema diff in a new tab.
- The label (Identical: 20, Different: 10, etc.) should change based on the selection of the Filter. It shows all by default and does not change.
- Overall font size in react data grid should be increased. The header should be in bold.
- Two vertical scroll bars are visible. Remove if possible.
- The components should be aligned properly after deselecting all the filters. Take a look at the existing behavior.
- Check the resize behavior against the existing one. Try to make it the same.
- In the DDL comparison panel, no DDL statement is displayed when the 'Identical' node is selected.
- On selection, remove the border of the cell if we are showing the selected row in a different color. Also, remove the border from the header node selection.
Hi Hackers,Please find attached the patch for RM-6133 : [React] Port schema diff to React.This patch also included the changes for removing SlickGrid from pgAdmin.--
Akshay Joshi Principal Software Architect +91 9767888246 | |
Hi NikhilThe review comments are below:
- Reduce some space from source selection and target selection.
- Inside the select control, make the server group label bold.
- There is currently a search icon instead of a filter icon. On the button, change the cursor pointer to none.
- Add a help button near the 'Filter' button.
- The spinner and comparison status string should be bolded or larger.
- The distance between the 'Generate Script' and 'Filter' buttons should be the same, it should not change on resizing of the browser. or open schema diff in a new tab.
- The label (Identical: 20, Different: 10, etc.) should change based on the selection of the Filter. It shows all by default and does not change.
- Overall font size in react data grid should be increased. The header should be in bold.
- Two vertical scroll bars are visible. Remove if possible.
- The components should be aligned properly after deselecting all the filters. Take a look at the existing behavior.
- Check the resize behavior against the existing one. Try to make it the same.
- In the DDL comparison panel, no DDL statement is displayed when the 'Identical' node is selected.
- On selection, remove the border of the cell if we are showing the selected row in a different color. Also, remove the border from the header node selection.
On Mon, Sep 5, 2022 at 4:09 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Hackers,Please find attached the patch for RM-6133 : [React] Port schema diff to React.This patch also included the changes for removing SlickGrid from pgAdmin.----
Akshay Joshi
Principal Software Architect
+91 9767888246
Attachment
Hi Akshay,PFA updated patch, I have resolved the review comments, there is one issue with resize window I will send another patch for it.On Mon, Sep 5, 2022 at 6:34 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Hi NikhilThe review comments are below:
- Reduce some space from source selection and target selection.
- Inside the select control, make the server group label bold.
Only this review comment is remaining, I will try to add it with a resize window patch.
- There is currently a search icon instead of a filter icon. On the button, change the cursor pointer to none.
- Add a help button near the 'Filter' button.
- The spinner and comparison status string should be bolded or larger.
- The distance between the 'Generate Script' and 'Filter' buttons should be the same, it should not change on resizing of the browser. or open schema diff in a new tab.
- The label (Identical: 20, Different: 10, etc.) should change based on the selection of the Filter. It shows all by default and does not change.
- Overall font size in react data grid should be increased. The header should be in bold.
- Two vertical scroll bars are visible. Remove if possible.
- The components should be aligned properly after deselecting all the filters. Take a look at the existing behavior.
- Check the resize behavior against the existing one. Try to make it the same.
- In the DDL comparison panel, no DDL statement is displayed when the 'Identical' node is selected.
- On selection, remove the border of the cell if we are showing the selected row in a different color. Also, remove the border from the header node selection.
On Mon, Sep 5, 2022 at 4:09 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Hackers,Please find attached the patch for RM-6133 : [React] Port schema diff to React.This patch also included the changes for removing SlickGrid from pgAdmin.----
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite
Akshay Joshi Principal Software Architect +91 9767888246 | |
Thanks, the patch applied.Please send another patch with updated screenshots and the issues you have mentioned.On Wed, Sep 7, 2022 at 6:58 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Akshay,PFA updated patch, I have resolved the review comments, there is one issue with resize window I will send another patch for it.On Mon, Sep 5, 2022 at 6:34 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Hi NikhilThe review comments are below:
- Reduce some space from source selection and target selection.
- Inside the select control, make the server group label bold.
Only this review comment is remaining, I will try to add it with a resize window patch.
- There is currently a search icon instead of a filter icon. On the button, change the cursor pointer to none.
- Add a help button near the 'Filter' button.
- The spinner and comparison status string should be bolded or larger.
- The distance between the 'Generate Script' and 'Filter' buttons should be the same, it should not change on resizing of the browser. or open schema diff in a new tab.
- The label (Identical: 20, Different: 10, etc.) should change based on the selection of the Filter. It shows all by default and does not change.
- Overall font size in react data grid should be increased. The header should be in bold.
- Two vertical scroll bars are visible. Remove if possible.
- The components should be aligned properly after deselecting all the filters. Take a look at the existing behavior.
- Check the resize behavior against the existing one. Try to make it the same.
- In the DDL comparison panel, no DDL statement is displayed when the 'Identical' node is selected.
- On selection, remove the border of the cell if we are showing the selected row in a different color. Also, remove the border from the header node selection.
On Mon, Sep 5, 2022 at 4:09 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Hackers,Please find attached the patch for RM-6133 : [React] Port schema diff to React.This patch also included the changes for removing SlickGrid from pgAdmin.----
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite--
Akshay Joshi
Principal Software Architect
+91 9767888246
Attachment
Hi Akshay,PFA patch to resolve following issues.Details:1. Resolved resize window issue for DDL result panel.2. Updated the Screenshots.3. Updated server group title to bold.On Wed, Sep 7, 2022 at 7:25 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Thanks, the patch applied.Please send another patch with updated screenshots and the issues you have mentioned.On Wed, Sep 7, 2022 at 6:58 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Akshay,PFA updated patch, I have resolved the review comments, there is one issue with resize window I will send another patch for it.On Mon, Sep 5, 2022 at 6:34 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Hi NikhilThe review comments are below:
- Reduce some space from source selection and target selection.
- Inside the select control, make the server group label bold.
Only this review comment is remaining, I will try to add it with a resize window patch.
- There is currently a search icon instead of a filter icon. On the button, change the cursor pointer to none.
- Add a help button near the 'Filter' button.
- The spinner and comparison status string should be bolded or larger.
- The distance between the 'Generate Script' and 'Filter' buttons should be the same, it should not change on resizing of the browser. or open schema diff in a new tab.
- The label (Identical: 20, Different: 10, etc.) should change based on the selection of the Filter. It shows all by default and does not change.
- Overall font size in react data grid should be increased. The header should be in bold.
- Two vertical scroll bars are visible. Remove if possible.
- The components should be aligned properly after deselecting all the filters. Take a look at the existing behavior.
- Check the resize behavior against the existing one. Try to make it the same.
- In the DDL comparison panel, no DDL statement is displayed when the 'Identical' node is selected.
- On selection, remove the border of the cell if we are showing the selected row in a different color. Also, remove the border from the header node selection.
On Mon, Sep 5, 2022 at 4:09 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Hackers,Please find attached the patch for RM-6133 : [React] Port schema diff to React.This patch also included the changes for removing SlickGrid from pgAdmin.----
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite--
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite
Akshay Joshi Principal Software Architect +91 9767888246 | |
- When some option from filter is selected, that particular type should be completely removed from top level result instead of showing identical:0
- Copy button is missing for Difference SQL
- Throw error when source & target server versions selected are different
- In High contrast - Font color for identical turns same as background.
- In High contrast - Checkbox box column turns same color of row resulting unable to distingiush about selection of item.
Thanks, the patch applied.On Thu, Sep 8, 2022 at 1:57 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Akshay,PFA patch to resolve following issues.Details:1. Resolved resize window issue for DDL result panel.2. Updated the Screenshots.3. Updated server group title to bold.On Wed, Sep 7, 2022 at 7:25 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Thanks, the patch applied.Please send another patch with updated screenshots and the issues you have mentioned.On Wed, Sep 7, 2022 at 6:58 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Akshay,PFA updated patch, I have resolved the review comments, there is one issue with resize window I will send another patch for it.On Mon, Sep 5, 2022 at 6:34 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Hi NikhilThe review comments are below:
- Reduce some space from source selection and target selection.
- Inside the select control, make the server group label bold.
Only this review comment is remaining, I will try to add it with a resize window patch.
- There is currently a search icon instead of a filter icon. On the button, change the cursor pointer to none.
- Add a help button near the 'Filter' button.
- The spinner and comparison status string should be bolded or larger.
- The distance between the 'Generate Script' and 'Filter' buttons should be the same, it should not change on resizing of the browser. or open schema diff in a new tab.
- The label (Identical: 20, Different: 10, etc.) should change based on the selection of the Filter. It shows all by default and does not change.
- Overall font size in react data grid should be increased. The header should be in bold.
- Two vertical scroll bars are visible. Remove if possible.
- The components should be aligned properly after deselecting all the filters. Take a look at the existing behavior.
- Check the resize behavior against the existing one. Try to make it the same.
- In the DDL comparison panel, no DDL statement is displayed when the 'Identical' node is selected.
- On selection, remove the border of the cell if we are showing the selected row in a different color. Also, remove the border from the header node selection.
On Mon, Sep 5, 2022 at 4:09 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Hackers,Please find attached the patch for RM-6133 : [React] Port schema diff to React.This patch also included the changes for removing SlickGrid from pgAdmin.----
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite--
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite--
Akshay Joshi
Principal Software Architect
+91 9767888246
Attachment
Hi Akshay,PFA patch for resolving the following issues reported in testing:
- When some option from filter is selected, that particular type should be completely removed from top level result instead of showing identical:0
- Copy button is missing for Difference SQL
- Throw error when source & target server versions selected are different
- In High contrast - Font color for identical turns same as background.
- In High contrast - Checkbox box column turns same color of row resulting unable to distingiush about selection of item.
Please ignore the previous patch, PFA updated patch included security fixes.On Thu, Sep 8, 2022 at 3:32 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Thanks, the patch applied.On Thu, Sep 8, 2022 at 1:57 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Akshay,PFA patch to resolve following issues.Details:1. Resolved resize window issue for DDL result panel.2. Updated the Screenshots.3. Updated server group title to bold.On Wed, Sep 7, 2022 at 7:25 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Thanks, the patch applied.Please send another patch with updated screenshots and the issues you have mentioned.On Wed, Sep 7, 2022 at 6:58 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Akshay,PFA updated patch, I have resolved the review comments, there is one issue with resize window I will send another patch for it.On Mon, Sep 5, 2022 at 6:34 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Hi NikhilThe review comments are below:
- Reduce some space from source selection and target selection.
- Inside the select control, make the server group label bold.
Only this review comment is remaining, I will try to add it with a resize window patch.
- There is currently a search icon instead of a filter icon. On the button, change the cursor pointer to none.
- Add a help button near the 'Filter' button.
- The spinner and comparison status string should be bolded or larger.
- The distance between the 'Generate Script' and 'Filter' buttons should be the same, it should not change on resizing of the browser. or open schema diff in a new tab.
- The label (Identical: 20, Different: 10, etc.) should change based on the selection of the Filter. It shows all by default and does not change.
- Overall font size in react data grid should be increased. The header should be in bold.
- Two vertical scroll bars are visible. Remove if possible.
- The components should be aligned properly after deselecting all the filters. Take a look at the existing behavior.
- Check the resize behavior against the existing one. Try to make it the same.
- In the DDL comparison panel, no DDL statement is displayed when the 'Identical' node is selected.
- On selection, remove the border of the cell if we are showing the selected row in a different color. Also, remove the border from the header node selection.
On Mon, Sep 5, 2022 at 4:09 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Hackers,Please find attached the patch for RM-6133 : [React] Port schema diff to React.This patch also included the changes for removing SlickGrid from pgAdmin.----
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite--
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite--
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite
Attachment
Hi Akshay,On Tue, Sep 13, 2022 at 3:09 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Akshay,PFA patch for resolving the following issues reported in testing:
- When some option from filter is selected, that particular type should be completely removed from top level result instead of showing identical:0
- Copy button is missing for Difference SQL
- Throw error when source & target server versions selected are different
- In High contrast - Font color for identical turns same as background.
- In High contrast - Checkbox box column turns same color of row resulting unable to distingiush about selection of item.
Please ignore the previous patch, PFA updated patch included security fixes.Please ignore the previous patch, PFA updated patch included security fixes.On Thu, Sep 8, 2022 at 3:32 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Thanks, the patch applied.On Thu, Sep 8, 2022 at 1:57 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Akshay,PFA patch to resolve following issues.Details:1. Resolved resize window issue for DDL result panel.2. Updated the Screenshots.3. Updated server group title to bold.On Wed, Sep 7, 2022 at 7:25 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Thanks, the patch applied.Please send another patch with updated screenshots and the issues you have mentioned.On Wed, Sep 7, 2022 at 6:58 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Akshay,PFA updated patch, I have resolved the review comments, there is one issue with resize window I will send another patch for it.On Mon, Sep 5, 2022 at 6:34 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Hi NikhilThe review comments are below:
- Reduce some space from source selection and target selection.
- Inside the select control, make the server group label bold.
Only this review comment is remaining, I will try to add it with a resize window patch.
- There is currently a search icon instead of a filter icon. On the button, change the cursor pointer to none.
- Add a help button near the 'Filter' button.
- The spinner and comparison status string should be bolded or larger.
- The distance between the 'Generate Script' and 'Filter' buttons should be the same, it should not change on resizing of the browser. or open schema diff in a new tab.
- The label (Identical: 20, Different: 10, etc.) should change based on the selection of the Filter. It shows all by default and does not change.
- Overall font size in react data grid should be increased. The header should be in bold.
- Two vertical scroll bars are visible. Remove if possible.
- The components should be aligned properly after deselecting all the filters. Take a look at the existing behavior.
- Check the resize behavior against the existing one. Try to make it the same.
- In the DDL comparison panel, no DDL statement is displayed when the 'Identical' node is selected.
- On selection, remove the border of the cell if we are showing the selected row in a different color. Also, remove the border from the header node selection.
On Mon, Sep 5, 2022 at 4:09 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Hackers,Please find attached the patch for RM-6133 : [React] Port schema diff to React.This patch also included the changes for removing SlickGrid from pgAdmin.----
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite--
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil Mohite--
Akshay Joshi
Principal Software Architect
+91 9767888246
Regards,Nikhil MohiteRegards,Nikhil Mohite
Akshay Joshi Principal Software Architect +91 9767888246 | |