public inbox for [email protected]
help / color / mirror / Atom feedFrom: Aditya Toshniwal <[email protected]>
To: Victoria Henry <[email protected]>
Cc: Ashesh Vashi <[email protected]>
Cc: Anthony Emengo <[email protected]>
Cc: Joao De Almeida Pereira <[email protected]>
Cc: Dave Page <[email protected]>
Cc: Akshay Joshi <[email protected]>
Cc: Murtuza Zabuawala <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Cc: Khushboo Vashi <[email protected]>
Subject: Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree
Date: Mon, 11 Jun 2018 11:14:44 +0530
Message-ID: <CAM9w-_npBypUeKZzSeppLOmpvGUJjDtUS7KsDF3eYapQM5hjuA@mail.gmail.com> (raw)
In-Reply-To: <CANxYE3+Dm+CJT0-+zuCfAfApno+ntRB0TV6ggWAda4b27S0FBA@mail.gmail.com>
References: <CAE+jja=Gdd032H7tpoZD2C0m2R7SnTZpHX_oPx2K2zGbaaW9yg@mail.gmail.com>
<CAFOhELcrZsRkwV4beOC7UT_HYvi9Rk=HFJ355g8Bj4bqyzMLog@mail.gmail.com>
<CAE+jjaneGOR_R7QvYnZt=khumRsyhnrzFRhpMG5-Q+rR9guLzQ@mail.gmail.com>
<CAKKotZROcoO_rtrrJUTvpfgSh1w4ig1ogQrkDrKSQHuKEL7CdQ@mail.gmail.com>
<CAE+jjakEZtn0ct128Xd+es5xCZ5eZLwoNZw6X0ue7oFB+a83oQ@mail.gmail.com>
<CAFOhELe_t_QGBbcYYm3Zrfwf3z9674PY5sy2QS46wHDTf-8DHA@mail.gmail.com>
<CAE+jja=u+W+FNaWGVs1KPZm1eYA_F=7ktomSbmsxOo-cjuOdBw@mail.gmail.com>
<CAE+jja=TZzMXOq4+t=dA+LfTGkW66urB37Tt9sEJ_UjAiymbbQ@mail.gmail.com>
<CAE+jjamSM_pBLNmYLn6-hWMiQvToceV2iPuME-eH5c+tgaYf7A@mail.gmail.com>
<CA+OCxowN15jg_DV4o+R+OSczyGHHccnJ1sPXE_XukYugaTKyyw@mail.gmail.com>
<CA+OCxozTMcEDKZEu9YCOsSBSed23Ka8hHiO5s6VF76KopbipNQ@mail.gmail.com>
<CAG7mmowrM3b8cTY+zcCAcVma8CKWHwXSF3=BUDcSFLpjH=jQBg@mail.gmail.com>
<CAE+jjanjQ295x0DgffT8OARRv8fge5KOPJpMCnLWe+j0jdqGJQ@mail.gmail.com>
<CAG7mmoym9838PVKdDD=469Bsx6AFBV1ThMK_dyitUmG26147Qg@mail.gmail.com>
<CAG7mmoy=5W3scnPs-TVpcu49F7gZU2qySPaakwfEdcg6rg2RMQ@mail.gmail.com>
<CAG8BBZOAFgw6iDseWTSEUC+tiwc0zCoQyycXGFWbW6ueftCggA@mail.gmail.com>
<CAG7mmoyirqdDGZnXcz5odMBnsOYY83Dj6Uw8gV59OPTJSnaMAg@mail.gmail.com>
<CA+OCxoz3C+HRDzh05SKhMjSZPnHezf=F0eiNoo_HHqUtiJWPiQ@mail.gmail.com>
<CAG7mmoxjfZGA8Gp4HMByTAf6ZLZ7Oaj2yTUrgbOrpJ0H7OtQNQ@mail.gmail.com>
<CAE+jja=GFVqodrWJ+vzBYO4tzoZipKqdThKUqMOGyoZPtJc1+Q@mail.gmail.com>
<CAG7mmozE+8pneLOp8RokHZiav0kcM2Ts6NkHwrz2n31-Ykn_kA@mail.gmail.com>
<CA+OCxowNhQ7waRSPp5P--dFisLFkf+EEOwH+y0+t+ZT9dXLXTA@mail.gmail.com>
<CAE+jjamkYzDxjvSmdkU-1Gyys1FXtrXGr+d7=yrsAREmXsteOQ@mail.gmail.com>
<CAE+jja=vaQOE6XbJH34+N24CZY1DRdYz8wgkxxTx5KSaqw7pLQ@mail.gmail.com>
<CAE+jjakrO-RaK=m5YYUBrF=9SgYjOyU59bZo+vWxDoFXG75Jvg@mail.gmail.com>
<CAE+jjanc7WbL6k42=Dp8NBw9mdiv4jccK7t5JfN2rD2n8NiYTg@mail.gmail.com>
<CAFOhELfyS404C50JWdgV16sBO7WYAJYYOD80kBA1AEhREALCmw@mail.gmail.com>
<CANxoLDfyOFt2QDAXE5BVC6X8tHZV4q3K=_gRiuWN_ife9FYU_w@mail.gmail.com>
<CAE+jjamBMWfWCypF_h5x0NThhmGHx7to6HsJ66WSmJrL6UdQ+g@mail.gmail.com>
<CAG7mmoyjWCTZUXH-CBTpgNy4ErA_6ojRgd+dT63zdAkGMY_2JQ@mail.gmail.com>
<CAG8BBZOKQDKtyu1-2HVCP5HFy1rKFdPVrMTY0SPEhb8x7Q7PBA@mail.gmail.com>
<CAG7mmowkNtkoLGQFeWAH-TSOi+QWbHT2-RbMmjRDAMAxMufmeg@mail.gmail.com>
<CAE+jja=PLKUdA43dAz2exgLcy6n5EGbM0E4K1ZdKmi5hex7Rew@mail.gmail.com>
<CAG7mmoyxBz+D+933L9UzgQrmRgn3xHxHuW06fwsNr-WkgXD_8w@mail.gmail.com>
<CA+OCxoz3eSNJ25sy_AmvERjNp24pD=Dfq-3a4ZQrmBVedDf-4g@mail.gmail.com>
<CAG7mmoxqA-Wa4hi0nwwg9nQKo1zmvYwfLrnGvVJjmobsNCCYmQ@mail.gmail.com>
<CA+OCxoz_V+z3WrfsbxXWNYcFHCqaURRwiJb6R_N1Yn0PWZcd8A@mail.gmail.com>
<CAG7mmozzeu=oRrYv+QYR3u6jWUfj7rZMzTbF23w3cB7WEkY7fA@mail.gmail.com>
<CAG7mmoxk6mzxxNm1YKU6=wKJ9KTAf4gKYX0y0AJxh6pS1h6HCw@mail.gmail.com>
<CAE+jjakrT2AhM2gEGt+QKEANMypn08O4vweOgwhyqQdV6mZEEQ@mail.gmail.com>
<CAG7mmowDFFbPFSxn0swpM2Ub_W0x842Yz=qiBmAC8ZxPYtn+Rw@mail.gmail.com>
<CAG8BBZNndOje9T-B=Jrxj0tqJbDYc489yYbkJkJx_V5LKzsM-w@mail.gmail.com>
<CAG7mmoxmwsd9GOtvAw-eT6=vT5JEx0VJYVf1H97Tk57khezOwA@mail.gmail.com>
<CAE+jjandjswWj030ULtEvX9dnwTLzCdJ=NY82H7BHPEQjRNsfQ@mail.gmail.com>
<CAG7mmozES8twvhLDKDJfHTp1F+xvvOt4qWmjXNTsPqJnPY=mjg@mail.gmail.com>
<CAG8BBZPXP9Z+NuQ20S_cctjrudZYbCDahu-4aFLpyh+RBeuUwA@mail.gmail.com>
<CANxYE3KtK8Ot=ddQHChD3it74uoQjOcPUS23=YkW+5Lzo3V60Q@mail.gmail.com>
<CAG7mmox1C7XM23wKhhG0ok8wkKhh2QHy1EsNrmKJS3eR29KCMg@mail.gmail.com>
<CANxYE3JMSG8Wvo_2LfrqpBk0KvfrzPg3R5hDhXChS9dM=iocdg@mail.gmail.com>
<CAM9w-_nVa3n3ZrWPS5h5ec_ZP+kxQ6HMWPJY9XCHsOkaszFSBw@mail.gmail.com>
<CANxYE3+_mNAwaZ6v9cCsF4jOY2sbPZTFiCuO1MgXUobuVFbxyw@mail.gmail.com>
<CAM9w-_ki9n+kxupnyYfqmcRE7ik96bd-FfeiponEr+husY6g4w@mail.gmail.com>
<CANxYE3+Dm+CJT0-+zuCfAfApno+ntRB0TV6ggWAda4b27S0FBA@mail.gmail.com>
Hi Victoria,
On Thu, Jun 7, 2018 at 8:51 PM, Victoria Henry <[email protected]> wrote:
> Hi Aditya
>
> Sure. I did not find moving web/pgadmin/tools/datagrid/static/js/datagrid.js.
>> Please correct me if I am missing anything.
>
> Generally speaking, I agree with moving/deleting files if it makes sense.
> But in regards to web/pgadmin/tools/datagrid/static/js/datagrid.js, it
> looks like this is still being used in web/pgadmin/tools/sqleditor/
> static/js/sqleditor.js with Datagrid.create_transaction
>
And this is another js which is not moved.(sqleditor.js). If we are moving
js files, why not this too.
> Sincerely,
>
> Victoria
>
>
>
>
> On Thu, Jun 7, 2018 at 12:35 AM Aditya Toshniwal <aditya.toshniwal@
> enterprisedb.com> wrote:
>
>> Hi Victoria,
>>
>> On Wed, Jun 6, 2018 at 8:55 PM, Victoria Henry <[email protected]> wrote:
>>
>>> Hi Aditya,
>>>
>>> 1) Why don't we start using webpack alias's instead of using absolute
>>>> path. For eg,
>>>> import {RestoreDialogWrapper} from '../../../pgadmin/static/js/
>>>> restore/restore_dialog_wrapper';
>>>> can be used as import {RestoreDialogWrapper} from
>>>> 'pgadmin_static/js/restore/restore_dialog_wrapper';
>>>> by adding pgadmin_static alias to webpack config.
>>>
>>>
>>> Great point. In some areas of the code, we began making this change.
>>> There is already an alias in webpack shims for `
>>> ../../../pgadmin/static/js` called `sources`. You can find an example
>>> of this in import statements for `supported_database_node.js`
>>>
>>> 2) Few of the js are left behind, the ones which are used in python
>>>> __init__.py. Can't we move them too ? It would be nicer to not to leave
>>>> behind a single js.
>>>
>>> I'm not sure what you mean. Could you point to an example of a single
>>> js file?
>>>
>>
>> Sure. I did not find moving web/pgadmin/tools/datagrid/static/js/datagrid.js.
>> Please correct me if I am missing anything.
>>
>>>
>>> Sincerely,
>>>
>>> Victoria
>>>
>>> On Wed, Jun 6, 2018 at 7:07 AM Aditya Toshniwal <aditya.toshniwal@
>>> enterprisedb.com> wrote:
>>>
>>>> Hi Anthony/Victoria/Joao,
>>>>
>>>> I know I am very late to review on patch 004. The idea of moving js
>>>> files from tools to static folder looks good, but I have a few suggestions:
>>>>
>>>> 1) Why don't we start using webpack alias's instead of using absolute
>>>> path. For eg,
>>>> import {RestoreDialogWrapper} from '../../../pgadmin/static/js/
>>>> restore/restore_dialog_wrapper';
>>>> can be used as import {RestoreDialogWrapper} from
>>>> 'pgadmin_static/js/restore/restore_dialog_wrapper';
>>>> by adding pgadmin_static alias to webpack config.
>>>>
>>>> 2) Few of the js are left behind, the ones which are used in python
>>>> __init__.py. Can't we move them too ? It would be nicer to not to leave
>>>> behind a single js.
>>>>
>>>> Kindly let me know your views on this.
>>>>
>>>>
>>>> Thanks and Regards,
>>>> Aditya Toshniwal
>>>> Software Engineer | EnterpriseDB Software Solutions | Pune
>>>> "Don't Complain about Heat, Plant a tree"
>>>>
>>>> On Sat, Jun 2, 2018 at 12:47 AM, Victoria Henry <[email protected]>
>>>> wrote:
>>>>
>>>>> Hey Ashesh,
>>>>>
>>>>> LGTM! The some of the CI tests failed but it looks like a flake. But
>>>>> you can go ahead and merge this.
>>>>>
>>>>> Sincerely,
>>>>>
>>>>> Victoria
>>>>>
>>>>> On Fri, Jun 1, 2018 at 2:36 PM Ashesh Vashi <
>>>>> [email protected]> wrote:
>>>>>
>>>>>> On Fri, Jun 1, 2018 at 10:09 PM, Victoria Henry <[email protected]>
>>>>>> wrote:
>>>>>>
>>>>>>> Hi Ashesh,
>>>>>>>
>>>>>>> We just attempted to apply your patch over master but it did not
>>>>>>> work. We don't want to introduce any bugs or break any functionality.
>>>>>>> Please update the patch to make sure it is synced up with the master branch.
>>>>>>>
>>>>>> Please find the updated patch.
>>>>>>
>>>>>>>
>>>>>>> Sincerely,
>>>>>>>
>>>>>>> Victoria
>>>>>>>
>>>>>>> On Fri, Jun 1, 2018 at 11:18 AM Anthony Emengo <[email protected]>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> Hey Ashesh,
>>>>>>>>
>>>>>>>> Thanks for the explanation. It was great and it really helped!
>>>>>>>>
>>>>>>>> C pgadmin/browser/server_groups/servers/databases/schemas/static/js/child.js
>>>>>>>> C pgadmin/browser/server_groups/servers/databases/schemas/static/js/schema_child_tree_node.js
>>>>>>>>
>>>>>>>> It makes sense to remove duplication by extracting these attributes
>>>>>>>> out and setting the canDrop and canCreate functions here. But is
>>>>>>>> it possible to combine these two files into one since they are related so
>>>>>>>> we don’t need to import schema_child_tree_node?
>>>>>>>>
>>>>>>> That was the original plan, but 'pgadmin/browser/static/js//node.js'
>>>>>> script has too many dependecies, which are not easily portable.
>>>>>> And - that may lead to change the scope of the patch.
>>>>>>
>>>>>> Hence - I decided to use the separate file to make sure we have
>>>>>> enough test coverage (which is more imprortant than changing the scope).
>>>>>>
>>>>>>> M pgadmin/static/js/tree/tree.js
>>>>>>>>
>>>>>>>> The creation of the ancestorNode function feels like a
>>>>>>>> pre-optimization. That function is not used any where outside of the
>>>>>>>> tree.js file, so it’s more confusing to have it defined.
>>>>>>>>
>>>>>>> It is being used in the latest changes. :-)
>>>>>>
>>>>>>
>>>>>>> On a lighter note, could we avoid the !! syntax when possible? For
>>>>>>>> example, instead of return !!obj, we could do something like return
>>>>>>>> obj === undefined or return _.isUndefined(obj) as this is more
>>>>>>>> intuitive.
>>>>>>>>
>>>>>>>> https://softwareengineering.stackexchange.com/a/80092
>>>>>>>>
>>>>>>> I am kind of disagree here. But - I have changed it anyway.
>>>>>>
>>>>>>> In addition, please update this patch as it is out of sync with the
>>>>>>>> latest commit on the master branch. Otherwise, everything looks good!
>>>>>>>>
>>>>>>> Here - you go!
>>>>>>
>>>>>> -- Thanks, Ashesh
>>>>>>
>>>>>>>
>>>>>>>>
>>>>>>>> Thanks
>>>>>>>> Anthony && Victoria
>>>>>>>>
>>>>>>>> On Fri, Jun 1, 2018 at 7:52 AM Ashesh Vashi <
>>>>>>>> [email protected]> wrote:
>>>>>>>>
>>>>>>>>> On Thu, May 24, 2018 at 8:13 PM, Joao De Almeida Pereira <
>>>>>>>>> [email protected]> wrote:
>>>>>>>>>
>>>>>>>>>> Hey, Thanks so much for the reply.
>>>>>>>>>>
>>>>>>>>>> We've noticed that you've made several modifications on top of
>>>>>>>>>> our original patch. Unfortunately, we've found it very hard to follow.
>>>>>>>>>> Could we please get a brief synopsis of the changes you have made - just so
>>>>>>>>>> we can better understand the rationale behind them? Just like we've done
>>>>>>>>>> for you previously.
>>>>>>>>>>
>>>>>>>>> Please find the changes from your original patch:
>>>>>>>>>
>>>>>>>>> M webpack.shim.js
>>>>>>>>> M webpack.test.config.js
>>>>>>>>> - In order to specify the fake_browser in regression tests, we need to use 'pgbrowser/browser' in the 'schema_child_tree_node.js' script.D pgadmin/browser/server_groups/servers/databases/schemas/static/js/can_drop_child.js
>>>>>>>>> - We don't need this with the new implementation.C pgadmin/browser/server_groups/servers/databases/schemas/static/js/child.js
>>>>>>>>> - All the children of schema node have common properties as 'parent_type', 'canDrop', 'canDropCascase', 'canCreate'.
>>>>>>>>> Hence - instead of defining them in each node, we have created a base node, which will have all these properties.
>>>>>>>>> And, modified all schema children node to inherit from it.C pgadmin/browser/server_groups/servers/databases/schemas/static/js/schema_child_tree_node.js
>>>>>>>>> - In this script, we're defining three functions 'childCreateMenuEnabled', 'isTreeItemOfChildOfSchema', & 'isTreeNodeOfSchemaChild', which are used by the 'SchemaChildNode' objects.M pgadmin/browser/static/js/collection.js
>>>>>>>>> - Fixed an issue related to wrongly defined 'error' function for the Collection object.D pgadmin/static/js/menu/can_create.js
>>>>>>>>> - It defined the function, which was defining a check for creation of a schema child node, or not by looking at the parent node (i.e. a schema/catalog node).
>>>>>>>>> The file was not defintely placed under the wrong directory, because - the similar logic was under 'can_drop_child.js', and it was defined under 'pgadmin/browser/server_groups/servers/databases/schemas/static/js' directory.D pgadmin/static/js/menu/menu_enabled.jsC pgadmin/static/js/nodes/supported_database_node.js
>>>>>>>>> - Used by the external tools for checking whether the 'selected' tree-node is:
>>>>>>>>> + 'database' node, and it is allowed to connect it.
>>>>>>>>> + Or, it is one of the schema child (and, not 'catalog' child).
>>>>>>>>> - Finding the correct location was difficult for this, as there is no defined pattern, also it can be used by other functions too. Hence - moved it out of 'pgadmin/static/js/menu' directory.M pgadmin/static/js/tree/tree.js
>>>>>>>>> - Introduced a function, which returns the ancestor node object, fow which the condition is true.D regression/javascript/menu/can_create_spec.js
>>>>>>>>> D regression/javascript/menu/menu_enabled_spec.js
>>>>>>>>> D regression/javascript/schema/can_drop_child_spec.jsC regression/javascript/fake_browser/browser.js
>>>>>>>>> C regression/javascript/nodes/schema/child_menu_spec.js
>>>>>>>>> - Modified the regression to test the new functionalies.M pgadmin/browser/server_groups/servers/databases/schemas/**/*.js
>>>>>>>>> - Extending the schema child nodes from the 'SchemaChildNode' class defined in 'pgadmin/.../schemas/static/js/child.js' script.
>>>>>>>>>
>>>>>>>>> Let me know if you need more information.
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>> Let's keep in mind that the original intent was simply to
>>>>>>>>>> introduce this abstraction into the code base, which is a big enough task.
>>>>>>>>>> I'd hate for the scope of the changes we're making to expand beyond that.
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>> I have the mutual feeling.
>>>>>>>>>
>>>>>>>>> -- Thanks, Ashesh
>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Thanks
>>>>>>>>>> Joao && Anthony
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> On Thu, May 24, 2018 at 2:59 AM Ashesh Vashi <
>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>
>>>>>>>>>>> Sorry for the late reply.
>>>>>>>>>>> On Wed, May 16, 2018 at 8:55 PM, Anthony Emengo <
>>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>>
>>>>>>>>>>>> export function canCreate(pgBrowser, childOfCatalogType) {
>>>>>>>>>>>> return canCreateObject.bind({
>>>>>>>>>>>> browser: pgBrowser,
>>>>>>>>>>>> childOfCatalogType: childOfCatalogType,
>>>>>>>>>>>> });
>>>>>>>>>>>> }
>>>>>>>>>>>>
>>>>>>>>>>>> With respect to the above code: this bind pattern looks good
>>>>>>>>>>>> and seems like the idiomatic way to handle this in JavaScript. On a lighter
>>>>>>>>>>>> node, I don’t even see the need for an additional method to wrap it. The
>>>>>>>>>>>> invocation could have easily been like canCreate:
>>>>>>>>>>>> canCreateObject.bind({ browser: pgBrowser, childOfCatalogType:
>>>>>>>>>>>> childOfCatalogType }), I don’t feel too strongly here.
>>>>>>>>>>>>
>>>>>>>>>>> I do agree - we can handle the same problem many ways.
>>>>>>>>>>> I prefer object oriented pardigm more in general.
>>>>>>>>>>> Any way - I have modified the code with some other changes.
>>>>>>>>>>>
>>>>>>>>>>>> I renamed it as isValidTreeNodeData, because - we were using it
>>>>>>>>>>>> in for testing the tree data. Please suggest me the right place, and name.
>>>>>>>>>>>>
>>>>>>>>>>>> We’re not sure; maybe after continued refactoring, we will come
>>>>>>>>>>>> across more generic functions. At that point we can revisit this and create
>>>>>>>>>>>> a utils.js file.
>>>>>>>>>>>>
>>>>>>>>>>> Sure.
>>>>>>>>>>>
>>>>>>>>>>>> The original patch was separating them in different places, but
>>>>>>>>>>>> - still uses some of the functionalities directly from the tree, which was
>>>>>>>>>>>> happening because we have contextual menu.
>>>>>>>>>>>> To give a better solution, I can think of putting the menus
>>>>>>>>>>>> related code understand ‘sources/tree/menu’ directory.
>>>>>>>>>>>>
>>>>>>>>>>>> We’re particularly worried because we’re trying to avoid the
>>>>>>>>>>>> coupling that we see in the code base today. We want to decouple *application
>>>>>>>>>>>> state* from *business domain* logic as much as we can -
>>>>>>>>>>>> because this makes the code much easier to understand. We achieve lower
>>>>>>>>>>>> coupling by have more suitable interfaces to retrieve *application
>>>>>>>>>>>> state* like: anyParent (the menu doesn’t care how this
>>>>>>>>>>>> happens). This is the direction that we’re trying to move towards, we just
>>>>>>>>>>>> don’t want the package structure to undermine that developer intent.
>>>>>>>>>>>>
>>>>>>>>>>> I realized after revisiting the code, menu/can_create.js was
>>>>>>>>>>> only applicable to the children of the schema/catalog nodes, same as
>>>>>>>>>>> 'can_drop_child'.
>>>>>>>>>>> We should have put both scripts in the same directory.
>>>>>>>>>>>
>>>>>>>>>>> Please find the updated patch for the same.
>>>>>>>>>>>
>>>>>>>>>>> Please review it, and let me know your concerns.
>>>>>>>>>>>
>>>>>>>>>>> -- Thanks, Ashesh
>>>>>>>>>>>
>>>>>>>>>>>> How about nodeMenu.isSupportedNode(…)?
>>>>>>>>>>>>
>>>>>>>>>>>> Naming is one of the hardest problems in programming. I don’t
>>>>>>>>>>>> feel too strongly about this one. For now, let’s keep it as is
>>>>>>>>>>>>
>>>>>>>>>>>> Thanks
>>>>>>>>>>>> Anthony && Victoria
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>
>>>>>>
>>>>
>>
view thread (69+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree
In-Reply-To: <CAM9w-_npBypUeKZzSeppLOmpvGUJjDtUS7KsDF3eYapQM5hjuA@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox