Fix: Make "Apply Annual Fees" currency-agnostic for non-INR deployments#3552
Fix: Make "Apply Annual Fees" currency-agnostic for non-INR deployments#3552Manishnemade12 wants to merge 2 commits intoopenMF:devfrom
Conversation
…nd loans views to prevent runtime errors
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
Note
|
Description
This PR fixes a critical internationalization (i18n) bug where the "Apply Annual Fees" menu option was completely hidden for any Fineract deployment not using the exact
<charge-name> - INRconvention (such as non-Rupee currencies like USD, EUR, KES, NGN, or localized charge names).Five occurrences across the Savings, Shares, and Recurring Deposits modules were hardcoded to check
charge.name === 'Annual fee - INR'. This string-matching brittle detection model was replaced with the stable, currency-agnostic Fineract API checkcharge.chargeTimeType?.id === 7. This conforms to Apache Fineract's global charge type enumeration for Annual Fees, ensuring the feature works accurately regardless of location, language, or currency.Screenshots, if any
(Not applicable, business logic fix enabling hidden dropdown options for localized currencies)
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!
If you have multiple commits please combine them into one commit by squashing them.
Read and understood the contribution guidelines at
web-app/.github/CONTRIBUTING.md.