-
-
Notifications
You must be signed in to change notification settings - Fork 789
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][IMP] report_async: add schedule_date field #904
[12.0][IMP] report_async: add schedule_date field #904
Conversation
Hi @kittiu, |
38a1f45
to
235305e
Compare
84877c7
to
9492918
Compare
9492918
to
f98265e
Compare
report_async/models/report_async.py
Outdated
@@ -71,7 +70,7 @@ class ReportAsync(models.Model): | |||
help="List all files created by this report background process", | |||
) | |||
|
|||
schedule_time = fields.Char(string='Schedule time') | |||
schedule_time = fields.Datetime(string='Schedule Time') |
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.
This would need migration scripts. Can we adapt the code in a way to avoid this change?
7993419
to
27e4d3b
Compare
This PR has the |
/ocabot merge major |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 675d7ec. Thanks a lot for contributing to OCA. ❤️ |
Change schedule_time to Datetime field
@ForgeFlow