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

FINERACT-1806: Advanced Charge-off Expense Accounting - "Advanced Accounting Rule" takes priority (if configured) #4190

Conversation

kulminsky
Copy link
Contributor

@kulminsky kulminsky commented Nov 26, 2024

Advanced Charge-off Expense Accounting - "Advanced Accounting Rule" takes priority (if configured)

Description

Advanced Charge-off Expense Accounting - "Advanced Accounting Rule" takes priority (if configured)

Ignore if these details are present on the associated Apache Fineract JIRA ticket.

Checklist

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Write the commit message as per https://github.com/apache/fineract/#pull-requests

  • Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

  • Create/update unit or integration tests for verifying the changes made.

  • Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

  • Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

  • Submission is not a "code dump". (Large changes can be made "in repository" via a branch. Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@kulminsky kulminsky force-pushed the FINERACT-1806/Advanced_Charge_off_Expense_Accounting_Advanced_Accounting_Rule_takes_priority branch from 865df2b to cb0bd43 Compare November 27, 2024 12:23
Copy link
Contributor

@galovics galovics left a comment

Choose a reason for hiding this comment

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

Let's add tests to cover the "priority".

@kulminsky kulminsky force-pushed the FINERACT-1806/Advanced_Charge_off_Expense_Accounting_Advanced_Accounting_Rule_takes_priority branch 3 times, most recently from d47add3 to 2299aa6 Compare December 4, 2024 11:40
@kulminsky kulminsky force-pushed the FINERACT-1806/Advanced_Charge_off_Expense_Accounting_Advanced_Accounting_Rule_takes_priority branch from 2299aa6 to 06868ee Compare December 4, 2024 14:18
@kulminsky kulminsky force-pushed the FINERACT-1806/Advanced_Charge_off_Expense_Accounting_Advanced_Accounting_Rule_takes_priority branch from 06868ee to cf3c0fd Compare December 4, 2024 14:22
Copy link
Contributor

@adamsaghy adamsaghy left a comment

Choose a reason for hiding this comment

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

LGTM

@adamsaghy adamsaghy merged commit 56ee2ea into apache:develop Dec 4, 2024
9 checks passed
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