New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[12.0][ADD] New module web_disable_browser_autocomplete #1623
[12.0][ADD] New module web_disable_browser_autocomplete #1623
Conversation
b497c3e
to
a5e4cd9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi renato.
Thanks for your contribution.
Some questions inline.
Regards.
@@ -0,0 +1,2 @@ | |||
# Copyright (C) 2020 Renato Lima - Akretion | |||
# License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you remove useless files ? Test, readme section...
@@ -0,0 +1,8 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> | |||
<templates id="template" xml:space="preserve"> | |||
<t t-extend="web.datepicker"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If i understand correctly, the auto complet will be disabled only for date field ? Is it normal ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@renatonlima resolve this from @legalsylvain please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correcty in this PR disable only for date fields, my original idea was disable for all input fields, but I don't found a way to do this, Do you have any suggestions for that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functional Test Ok. And i was feeling the need for something like this in date type fields.
Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes finally, this was an annoyance for years. Would be nice if this is also fixed on odoo/master
Please look at the comments of @legalsylvain
This PR has the |
@renatonlima Just remove /tests/init and empty readme stuff for cleanliness and i would be happy to merge this. |
7ae6748
to
2f4a20a
Compare
@gfcapalbo and travis is green |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correcty in this PR disable only for date fields, my original idea was disable for all input fields, but I don't found a way to do this, Do you have any suggestions for that?
could you just add a text in the roadmap section. for the time being the description is not correct. says that disable autocomplete, but in fact only for date field.
otherwise, LGTM .Thanks !
2f4a20a
to
a7e287e
Compare
@legalsylvain Done, Thanks for your review! |
@gfcapalbo could you merge? |
ping @OCA/web-maintainers |
/ocabot merge patch |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 4e45f6a. Thanks a lot for contributing to OCA. ❤️ |
This module allows you to disable the autocomplete function of the web browser for Odoo input fields.