Update Fineract

[Pinchez25]

What is the Purpose?

What was the approach?

How can the changes made get tested?

Are there any concerns to addressed further before or after merging this PR?

Mentions?

Issue(s) affected?

[willymwai]

/review

[willymwai]

/improve

[willymwai]

/describe

[willymwai]

/review

[willymwai]

/improve

[qodo-code-review]

PR Code Suggestions ✨

Category	Suggestion	Impact
General	Pin image by digest Pin the image by digest to prevent accidental drift if the tag is mutable, ensuring deterministic deployments. Add digest and keep tag for readability but prefer digest pull policy. charts/jisort/values.yaml [106-108] image: name: truehostcloud/fineract tag: 353708 + digest: sha256:REPLACE_WITH_ACTUAL_DIGEST Apply / Chat Suggestion importance[1-10]: 6 __ Why: This is a valid best practice suggestion that improves deployment determinism and security by pinning the image version via its digest.	Low
More

[qodo-code-review]

PR Reviewer Guide 🔍

(Review updated until commit 04dd833)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Image Tag Change Verify that the new image tag 353708 exists, is immutable, and corresponds to the intended Fineract version 1.18.1; ensure no breaking config changes are required for this image. name: truehostcloud/fineract tag: 353708 resources: Version Consistency Confirm that chart version: 0.4.1 and appVersion: "1.18.1" align with release notes and dependent manifests; update changelog and docs accordingly. version: 0.4.1 # This is the version number of the application being deployed. This version number should be # incremented each time you make changes to the application. Versions are not expected to # follow Semantic Versioning. They should reflect the version the application is using. # It is recommended to use it with quotes. appVersion: "1.18.1"

[qodo-code-review]

Persistent review updated to latest commit 04dd833

[qodo-code-review]

User description

What is the Purpose?

What was the approach?

How can the changes made get tested?

Are there any concerns to addressed further before or after merging this PR?

Mentions?

Issue(s) affected?

---

PR Type

Enhancement

---

Description

• Update Fineract application version from 1.18.0 to 1.18.1

• Bump chart version from 0.4.0 to 0.4.1

• Update Docker image tag from 63ee17 to 353708

---

Diagram Walkthrough

flowchart LR
  A["Chart v0.4.0"] --> B["Chart v0.4.1"]
  C["Fineract v1.18.0"] --> D["Fineract v1.18.1"]
  E["Image tag 63ee17"] --> F["Image tag 353708"]

Loading

File Walkthrough

Relevant files

Enhancement

Chart.yaml Update chart and app versions                                                        charts/jisort/Chart.yaml Increment chart version from 0.4.0 to 0.4.1 Update appVersion from 1.18.0 to 1.18.1 +2/-2      values.yaml Update Docker image tag                                                                    charts/jisort/values.yaml Update Fineract Docker image tag from 63ee17 to 353708 +1/-1

---

[qodo-code-review]

PR Code Suggestions ✨

Category	Suggestion	Impact
General	Pin image by digest Pin the image by digest to prevent accidental drift if the tag is mutable, ensuring deterministic deployments. Add digest and keep tag for readability but prefer digest pull policy. charts/jisort/values.yaml [106-108] image: name: truehostcloud/fineract tag: 353708 + digest: sha256:REPLACE_WITH_ACTUAL_DIGEST Apply / Chat Suggestion importance[1-10]: 6 __ Why: This is a valid best practice suggestion that improves deployment determinism and security by pinning the image version via its digest.	Low
More