Skip to content

Release mifos-v2.0.0: Closedloop batch fixes, JDK 17, batch status improvements#128

Open
tdaly61 wants to merge 24 commits into
masterfrom
dev
Open

Release mifos-v2.0.0: Closedloop batch fixes, JDK 17, batch status improvements#128
tdaly61 wants to merge 24 commits into
masterfrom
dev

Conversation

@tdaly61

@tdaly61 tdaly61 commented Mar 16, 2026

Copy link
Copy Markdown
Contributor

Summary

This release brings ph-ee-operations-app to version 2.0.0.mifos-SNAPSHOT with critical closedloop batch reporting fixes, Java 17 migration, and PHEE v1.13.3 integration.

Key Changes

  • Closedloop batch status fix (GAZ-253): evaluateBatchSummary() now falls back to parent batchId when no transfers found by sub-batch ID — fixes status=NULL / completed_at=NULL for closedloop batches where transfers are stored under parent batchId
  • NonUniqueResultException fix (GAZ-253): BatchRepository.findByBatchId replaced with findParentsByBatchId (returns List, ORDER BY id DESC) to handle duplicate parent rows without JPA exception
  • GovStack batch API fix: Fixed BatchApi.evaluateBatchSummary() for correct status aggregation in GovStack mode
  • PHEE v1.13.3 merge: Incorporated upstream operations-app changes
  • JDK 17 migration: Updated to eclipse-temurin:17-jdk; updated Dockerfile and CircleCI config
  • Mifos JFrog: Updated build.gradle for Mifos JFrog Artifactory dependencies
  • Version: Bumped to 2.0.0.mifos-SNAPSHOT

Test Plan

  • Submit closedloop batch — verify status=COMPLETED, completed_at set, correct totals, no duplicates in ops-web
  • Submit GovStack batch — verify parent batch aggregation correct
  • Verify no NonUniqueResultException in logs when duplicate rows transiently exist
  • Confirm CircleCI pipeline passes

🤖 Generated with Claude Code

tdaly61 and others added 24 commits April 3, 2025 23:00
[GAZ-253] bug fixes for closedloop batch handling, was reporting duplicates and…
[GAZ-253] fix: correct batch status and totals for closedloop transfers
[GAZ-258] fix: build.gradle had wrong description field, now corrected
@tdaly61
tdaly61 requested a review from DavidH-1 March 16, 2026 10:44
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