Skip to content

Conversation

@meval1006
Copy link
Contributor

@meval1006 meval1006 commented Jan 27, 2026

Removed the following sections:

  • Dashboard
  • Rental products
  • Pricing subsection
  • Reservations subsection

Added:

  • Card links to Physical and service rental products
  • Updated Settings section to saas-19.1
  • Upgraded and lightly revised the Price computing section to give more context in the example.

This saas-19.1 PR can be FWP up to master.

@meval1006 meval1006 self-assigned this Jan 27, 2026
@meval1006 meval1006 added the 2 label Jan 27, 2026
@robodoo
Copy link
Collaborator

robodoo commented Jan 27, 2026

Pull request status dashboard

@meval1006 meval1006 removed the request for review from samueljlieber January 27, 2026 02:03
@meval1006 meval1006 marked this pull request as ready for review January 27, 2026 02:04
@meval1006
Copy link
Contributor Author

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.

@C3POdoo C3POdoo requested a review from a team January 27, 2026 02:06
Copy link
Contributor

@huisit huisit left a 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:
Copy link
Contributor

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>
@meval1006 meval1006 force-pushed the saas-19.1-rental-update-homepage-meval branch from 0a6142f to 740f29e Compare January 28, 2026 02:40
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.

3 participants