-
Notifications
You must be signed in to change notification settings - Fork 11.1k
[IMP] rental: update homepage to v19.1 #16130
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
base: saas-19.1
Are you sure you want to change the base?
Conversation
|
Hi @huisit , this PR is ready for review. If you know how to fix the alignment for the math equation in the Price computing section, I am open to feedback. Thank you. |
huisit
left a comment
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.
@meval1006 wow you made this so much cleaner! I shared a formatting suggestion to add equality sign alignment for the math exercise, hope that helps to clarify the price computing rules.
| If a rental business has :ref:`multiple locations <rental/multi-location-management>`, rental | ||
| products can be transferred and tracked between them. | ||
|
|
||
| .. _rental/pricing: |
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.
Runbot is flagging a warning because this label was removed, and it's still referenced in sales/rental/manage_deposits.rst. You could add it back or change the reference in the other doc, up to your discretion!
Co-authored-by: Rex <11528309+huisit@users.noreply.github.com>
0a6142f to
740f29e
Compare

Removed the following sections:
Added:
This saas-19.1 PR can be FWP up to master.