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

[16.0][IMP] cooperator: Add specific income account for share increase #145

Open
wants to merge 2 commits into
base: 16.0
Choose a base branch
from

Conversation

nicolasramossomit
Copy link

This PR adds the capability to use a different income account for share increase operations in the cooperator module. Key changes:

  1. Added new company-dependent fields:

    • property_account_income_increase_id on product.template
    • property_account_income_increase_categ_id on product.category
  2. Modified invoice line preparation to use specific account for increase operations:

    • Uses new increase account when subscription type is 'increase'
    • Falls back to standard income account if increase account is not set
  3. Added corresponding tests:

    • Test for standard income account in new subscriptions
    • Test for increase income account in increase subscriptions

The change allows cooperatives to track income from share increases separately from initial share subscriptions, improving financial reporting and accounting flexibility.

Technical details:

  • Extends existing account fields pattern
  • Maintains backward compatibility
  • Fully tested with both scenarios

@nicolasramossomit nicolasramossomit changed the title [IMP] cooperator: Add specific income account for share increase [16.0][IMP] cooperator: Add specific income account for share increase Nov 13, 2024
@nicolasramossomit nicolasramossomit force-pushed the cooperator_voluntary branch 2 times, most recently from 89b467f to 5885fb2 Compare November 13, 2024 13:11
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.

2 participants