-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[16.0][ADD] web_widget_remaining_days_exact_date: New module to show exact date when remaining days shown #2916
[16.0][ADD] web_widget_remaining_days_exact_date: New module to show exact date when remaining days shown #2916
Conversation
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.
Tested. Tiny suggestion:
{ | ||
get diffString() { | ||
var value = this._super(...arguments); | ||
if (this.props.exact_date && Math.abs(this.diffDays) <= 99) { |
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.
Add comment on why the hardcoded 99 value :)
…date when remaining days shown This module is adding an option to the remaining_days widget to be able to show the exact date.
9cf8e43
to
a2f4298
Compare
I have changed the module's behavior so that this option works by default, and in case you don't want to use it, it can be disabled with options="{'exact_date': False}". Please review it again @chienandalu |
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.
👍
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.
/ocabot merge nobump
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 5cdc126. Thanks a lot for contributing to OCA. ❤️ |
This module is adding an option to the remaining_days widget to be able to show the exact date.
cc @Tecnativa TT50517
ping @pedrobaeza @chienandalu