Skip to content

Compare Upstream#5779

Draft
andrew-nkhoma wants to merge 85 commits intoapache:developfrom
andrew-nkhoma:develop
Draft

Compare Upstream#5779
andrew-nkhoma wants to merge 85 commits intoapache:developfrom
andrew-nkhoma:develop

Conversation

@andrew-nkhoma
Copy link
Copy Markdown

Description

Describe the changes made and why they were made. (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 our guidelines
  • 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 our 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
  • This PR must not be a "code dump". Large changes can be made in a branch, with assistance. Ask for help on the developer mailing list.

Your assigned reviewer(s) will follow our guidelines for code reviews.

Copilot AI and others added 30 commits January 30, 2026 11:07
Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
…ILIKE

Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
…arch-api

Add case-insensitive search and productId to Search API
Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
…-productid

Add productId filter to loans list endpoint
…tedDate

Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
…uery-issue

Fix ClassCastException in trial balance job query
Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
…upport

Add journal entry support to Batch API
…-for-teller-issues

Add Teller Cash Management redesign issue tracking: docs + creation script
…ebit-credit-accounts

Fix #17: PUT /tellers silently drops debitAccountId and creditAccountId
…raint-cashiers

Fix: Remove m_cashiers unique constraint on (staff_id, teller_id)
…ansaction params

Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
Agent-Logs-Url: https://github.com/andrew-nkhoma/fineract/sessions/c6880b26-04d8-4598-bfd0-3d93dca59ca3
…-parameters

Fix: fromDate/toDate query parameters ignored on cashier transactions endpoints
…ons-table

feat: Add m_cashier_sessions table and JPA entity (Issue #20)
Copilot AI and others added 25 commits March 25, 2026 11:57
…-out) in closeSession() and getSessionSummary()

Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
Agent-Logs-Url: https://github.com/andrew-nkhoma/fineract/sessions/bc25f20e-fc47-45f8-a71e-d20a9a27f071
…ansaction-types

Fix: align closeSession() variance calculation with getSessionSummary() txn type sets
…resolves #43)

Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
Agent-Logs-Url: https://github.com/andrew-nkhoma/fineract/sessions/234a5797-fe73-47a7-b483-facfb6124f39
…om-dotransactionforcashier

chore: remove dead entityType code block from doTransactionForCashier() (resolves #43)
…Cashier

Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
Agent-Logs-Url: https://github.com/andrew-nkhoma/fineract/sessions/850c6761-b280-445a-9a7d-cd1adf6b67c4
…allocate-cash

Add pre-validation to allocateCashToCashier to match settleCashFromCashier pattern
…Handler

Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
Agent-Logs-Url: https://github.com/andrew-nkhoma/fineract/sessions/e054108a-405d-494e-9513-1e1aa6882def
…e-platform-service

Remove CashierWritePlatformService stub and dead ModifyCashierCommandHandler
…ation for new permission codes

Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
Agent-Logs-Url: https://github.com/andrew-nkhoma/fineract/sessions/fc897cb5-3173-4071-8a4c-5d70f037afab
…-permissions

Add permission checks to CashierSessionApiResource endpoints
- CashierSessionReadPlatformServiceImpl: replace List.of(1, 101, 102) and
  List.of(2, 201, 202) with CashierTxnType.ALLOCATE/SETTLE/INWARD_CASH_TXN
  and OUTWARD_CASH_TXN constants; update explanatory comments
- TellerManagementReadPlatformServiceImpl: replace hardcoded 101, 103, 104
  literals in cashierTxnSummarySchema() SQL with CashierTxnType enum values;
  annotate remaining 105 (unclassified) with comment

Closes #47

Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
Agent-Logs-Url: https://github.com/andrew-nkhoma/fineract/sessions/705d2960-89ae-431e-8a31-d4a1e9434cd4
…-transaction-types

Replace hardcoded txn_type integers with CashierTxnType enum constants
… (issue #48)

Co-authored-by: andrew-nkhoma <95112149+andrew-nkhoma@users.noreply.github.com>
Agent-Logs-Url: https://github.com/andrew-nkhoma/fineract/sessions/63e307f7-b58e-4df6-afd7-c3c4b9518591
…y-parameter

Add optional sessionId query parameter to summaryandtransactions endpoint
…entry-reversal

Fix manual journal entry reversal to stop creating new offsetting entries
Copilot AI review requested due to automatic review settings April 19, 2026 17:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@vidakovic
Copy link
Copy Markdown
Contributor

@andrew-nkhoma ... missing reference to a Jira ticket in the PR title! (waaaaayyy) too many commits, please squash into 1. What problem are you trying to solve here? Seems like most of this was done by some AI tool... and it seems you have a bunch of unresolved conflicts. Can't really identify what kind of solution you are trying here to find... for that the number of changed files is too high (60+), and the changes are all over the place.

@vidakovic vidakovic marked this pull request as draft April 19, 2026 19:08
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.

4 participants