Skip to content

Conversation

@meval1006
Copy link
Contributor

SME requested the following changes:

  • Revised the Multi-location management and transfers section to provide a more precise explanation of what the Rental Transfers feature does.
  • Added new screenshots, a note, and an example for the section.
  • Moved the section closer to the Pickup products section.

Project task: https://www.odoo.com/odoo/action-4043/5351509

This 19.0 PR can be FWP up to master.

@meval1006 meval1006 requested a review from jero-odoo November 24, 2025 23:08
@meval1006 meval1006 self-assigned this Nov 24, 2025
@robodoo
Copy link
Collaborator

robodoo commented Nov 24, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team November 24, 2025 23:10
@meval1006
Copy link
Contributor Author

@jero-odoo This PR is ready for review when you have time. Thank you.

Copy link
Contributor

@jero-odoo jero-odoo left a comment

Choose a reason for hiding this comment

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

Hey @meval1006 , looks good! Just a few notes, let me know if you have any questions. Thanks!

@meval1006 meval1006 force-pushed the 19.0-rental-phyiscal-products-revision-meval branch from 7396dd7 to 6619440 Compare November 25, 2025 20:23
@meval1006
Copy link
Contributor Author

@StraubCreative Hi Zac, this PR is ready to merge. Thank you.

Copy link
Contributor

@StraubCreative StraubCreative left a comment

Choose a reason for hiding this comment

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

LGTM
@robodoo delegate+

@StraubCreative
Copy link
Contributor

@meval1006 considering adding a points value label on the PR before merging so it counts for the week/month 👍

@meval1006 meval1006 added the 2 label Nov 26, 2025
@meval1006
Copy link
Contributor Author

@StraubCreative, I have added points. Ready to merge. Thank you.

@meval1006 meval1006 added 3 and removed 2 labels Nov 26, 2025
@StraubCreative
Copy link
Contributor

@robodoo r+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants