[18.0][IMP] web_responsive: Quick Record Search #3192
Conversation
|
Thanks for this new feature. It seems interesting, but I wouldn't say this is something for the |
There was a problem hiding this comment.
Hi @mohammedshahil, thank you for your contribution!
Please check the comments below.
| @@ -497,6 +500,7 @@ <h2><a class="toc-backref" href="#toc-entry-6">Contributors</a></h2> | |||
| <li>Taras Shabaranskyi <<a class="reference external" href="mailto:shabaranskij@gmail.com">shabaranskij@gmail.com</a>></li> | |||
| <li>Angel Patel <<a class="reference external" href="mailto:patelangel1414@gmail.com">patelangel1414@gmail.com</a>></li> | |||
| <li>Mohamed Alkobrosli <<a class="reference external" href="mailto:malkobrosly@kencove.com">malkobrosly@kencove.com</a>></li> | |||
| <li>Mohammed Shahil <<a class="reference external" href="mailto:mohammedshahilmp@gmail.com">mohammedshahilmp@gmail.com</a>></li> | |||
There was a problem hiding this comment.
This file is auto-generated. Please modify the CONTRIBUTORS file instead. Check the example here.
There was a problem hiding this comment.
That's really good that you have added a screenshot! Please update the USAGE file and add the description of how this feature works in it. You can also embed this gif in it.
There was a problem hiding this comment.
@ivs-cetmix check my comment about the convenience of this feature being here: #3192 (comment)
There was a problem hiding this comment.
@ivs-cetmix check my comment about the convenience of this feature being here: #3192 (comment)
@pedrobaeza yup, that makes sense. I see two options:
- Make search in the same search string that is used now to search for apps (main screen). Then we need to update the web_responsive
- Leave it "as is". In this case yes, this should be a separate module.
@mohammedshahil what do you think?
There was a problem hiding this comment.
@ivs-cetmix check my comment about the convenience of this feature being here: #3192 (comment)
@pedrobaeza yup, that makes sense. I see two options:
* Make search in the same search string that is used now to search for apps (main screen). Then we need to update the web_responsive * Leave it "as is". In this case yes, this should be a separate module.@mohammedshahil what do you think?
yes separate module is better
Usage
Ctrl+K)./res.partner John/sale.order S00001/product.product Chairquick_record_search.mp4