-
Notifications
You must be signed in to change notification settings - Fork 23.2k
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
[FIX] web: Make datetimepicker configurable in datetime fields #27541
Conversation
c5368db
to
39dffc6
Compare
View: FormView, | ||
model: 'partner', | ||
data: this.data, | ||
arch:'<form string="Partners"><field name="datetime" options=\'{"datepicker": {"daysOfWeekDisabled": [0, 6]}}\'/></form>', |
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.
line too long
robodoo delegate=VincentSchippefilt |
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.
Cool! Could you please remove @Yajo
from the commit message? It spams me every time somebody clones that commit in another repo 😅
Just name me as Jairo Llopis, or Yajo without the @ please 🙏 , or link to the PR instead of me personally.
@yajo of course... |
e430f65
to
975a622
Compare
The datetimepicker option was introduced to be able to customize the datetime picker widget in date/datetime fields. However, due to the way the _makeDatePicker function was coded, it did not work in datetime fields. Thank to Yajo for the initial fix
robodoo r+ |
The datetimepicker option was introduced to be able to customize the datetime picker widget in date/datetime fields. However, due to the way the _makeDatePicker function was coded, it did not work in datetime fields. Thank to Yajo for the initial fix closes #27541
Merged in 078b31d |
This addon is no longer needed since odoo/odoo#25044 (comment) and odoo/odoo#27541, so I remove it.
This addon is no longer needed since odoo/odoo#25044 (comment) and odoo/odoo#27541, so I remove it.
The datetimepicker option was introduced to be able to customize the
datetime picker widget in date/datetime fields. However, due to the way
the _makeDatePicker function was coded, it did not work in datetime
fields.
Thank to @yajo for the initial fix
Description of the issue/feature this PR addresses:
Current behavior before PR:
Desired behavior after PR is merged:
--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr