feat: add MWK (Malawi Kwacha) payment methods#625
Conversation
Add an MWK entry to payment_methods.json with the payment rails Malawians actually use. Previously MWK had no entry and fell through to the generic default (Bank Transfer / Cash in person), so the app never suggested the country's dominant mobile-money rails. Airtel Money and TNM Mpamba are the two most widely used payment methods in Malawi, alongside bank transfers (incl. apps like Mo626) and cash.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughA single line is added to ChangesMWK Payment Methods Entry
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What
Adds an
MWK(Malawi Kwacha) entry toassets/data/payment_methods.jsonwith the payment methods actually used in Malawi: Airtel Money, TNM Mpamba, Bank Transfer, and Cash.Why
MWK had no entry in the file, so it fell through to the generic
default(Bank Transfer,Cash in person). I noticed this using the app from the Play Store when creating/browsing MWK orders, the only suggested methods were Bank Transfer and In Person.In practice, the two dominant ways Malawians send money are the mobile-money rails Airtel Money and TNM Mpamba, followed by bank transfers (including bank apps like Mo626). Without these, the suggestions don't match how people here actually pay, so users have to fall back to the custom field every time.
Change
A single line, placed alphabetically between
MLCandMXN, matching the file's existing single-line-per-currency format:
json "MWK": ["Airtel Money", "TNM Mpamba", "Bank Transfer", "Cash"], Testing
python3 -m json.tool) ✓I'm based in Malawi and use these rails myself, so happy to adjust naming/ordering if maintainers prefer a different convention.
Summary by CodeRabbit