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

[IMP] accounting: split deferred settings #11041

Open
wants to merge 1 commit into
base: saas-17.4
Choose a base branch
from

Conversation

RicGR98
Copy link
Contributor

@RicGR98 RicGR98 commented Sep 18, 2024

@robodoo
Copy link
Collaborator

robodoo commented Sep 18, 2024

This PR targets the disabled branch odoo/documentation:saas-17.3, it needs to be retargeted before it can be merged.

@robodoo
Copy link
Collaborator

robodoo commented Sep 18, 2024

Pull request status dashboard

@RicGR98 RicGR98 changed the base branch from saas-17.3 to saas-17.4 September 18, 2024 08:52
This has been done in the following commit
odoo/enterprise@39fffc5
@RicGR98 RicGR98 force-pushed the saas-17.3-accounting-split-deferred-rigr branch from 609f54a to 1b452ba Compare September 18, 2024 08:53
@C3POdoo C3POdoo requested a review from a team September 18, 2024 08:55
@dade-odoo
Copy link
Contributor

Hey @RicGR98 , as always thanks for keeping our accounting documentation up to date! I have two questions:

  1. In the Configuration sections of each doc, it looks like you removed the wrong lines for each - the deferred revenue doc includes the line explaining deferred expenses and vice versa. Was that a mistake or am I misunderstanding something?
  2. I see the value of Full Months is available for the Based on field. Could you add a brief explanation of how that would work (what makes it different from just Months)?
    Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants