Skip to content

[GAZ-253] fixes for govstack closedloop and mojaloop testing#171

Merged
DavidH-1 merged 1 commit into
devfrom
gaz-253
Mar 6, 2026
Merged

[GAZ-253] fixes for govstack closedloop and mojaloop testing#171
DavidH-1 merged 1 commit into
devfrom
gaz-253

Conversation

@tdaly61
Copy link
Copy Markdown

@tdaly61 tdaly61 commented Mar 6, 2026

Description

fixes for govstack closedloop and mojaloop from testing ARM and X64_86

    SendCallbackRoute.java — formatting/logic cleanup
    SendCallbackWorker.java — added List import
    application.yaml — max-retry: 10, delay: 30

Jira https://mifosforge.jira.com/browse/GAZ-253

  • PR title should have jira ticket enclosed in [].

    Format: [jira_ticket] description

    ex: [phee-123] PR title.
  • Add a link to the Jira ticket.
  • Describe the changes made and why they were made.

Checklist

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

  • Followed the PR title naming convention mentioned above.

  • Design related bullet points or design document link related to this PR added in the description above.

  • Updated corresponding Postman Collection or Api documentation for the changes in this PR.

  • Created/updated unit or integration tests for verifying the changes made.

  • Added required Swagger annotation and update API documentation with details of any API changes if applicable

  • Followed the naming conventions as given in https://docs.google.com/document/d/1Q4vaMSzrTxxh9TS0RILuNkSkYCxotuYk1Xe0CMIkkCU/edit?usp=sharing

    SendCallbackRoute.java — formatting/logic cleanup
    SendCallbackWorker.java — added List import
    application.yaml — max-retry: 10, delay: 30
@tdaly61 tdaly61 requested a review from DavidH-1 March 6, 2026 00:30
Copy link
Copy Markdown

@DavidH-1 DavidH-1 left a comment

Choose a reason for hiding this comment

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

lgtm

@DavidH-1 DavidH-1 merged commit 2032d34 into dev Mar 6, 2026
2 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.

2 participants