Skip to content
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

[15.0][FIX] sale_planner_calendar: 'calendar.event' object has no attribute 'start_datetime' #3225

Conversation

carolinafernandez-tecnativa
Copy link
Contributor

@carolinafernandez-tecnativa carolinafernandez-tecnativa commented Jul 5, 2024

  • With the router we can establish a recurring calendar event of type "Delivery" that means that when a sales order is confirmed that generates a delivery, it establishes the delivery date on the order on the next day that has been defined in the event. calendar.
  • Before this fix, when trying to confirm a sale order got this error message:
image

@Tecnativa
TT49977
@pedrobaeza @sergio-teruel

@pedrobaeza pedrobaeza added this to the 15.0 milestone Jul 6, 2024
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clear patch, so fast-tracking it:

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-3225-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 32373b9 into OCA:15.0 Jul 6, 2024
10 of 11 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at a92b11a. Thanks a lot for contributing to OCA. ❤️

@pedrobaeza pedrobaeza deleted the 15.0-fix-sale_planner_calendar-error-confirm-sale branch July 6, 2024 12:35
@pedrobaeza pedrobaeza changed the title [FIX] sale_planner_calendar: 'calendar.event' object has no attribute 'start_datetime' [15.0][FIX] sale_planner_calendar: 'calendar.event' object has no attribute 'start_datetime' Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants