Skip to content

feat: add MWK (Malawi Kwacha) payment methods#625

Merged
grunch merged 1 commit into
MostroP2P:mainfrom
codaMW:feat/add-mwk-payment-methods
Jun 22, 2026
Merged

feat: add MWK (Malawi Kwacha) payment methods#625
grunch merged 1 commit into
MostroP2P:mainfrom
codaMW:feat/add-mwk-payment-methods

Conversation

@codaMW

@codaMW codaMW commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

What

Adds an MWK (Malawi Kwacha) entry to assets/data/payment_methods.json with 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.

1000125745

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 MLC and MXN, matching the file's existing single-line-per-currency format:

json "MWK": ["Airtel Money", "TNM Mpamba", "Bank Transfer", "Cash"], ​

1000125672

Testing

  • JSON validated (python3 -m json.tool) ✓
  • Entry placed alphabetically, format matches surrounding entries ✓
  • Built and ran on a physical device (Nokia C31, Android 12); confirmed the create-order screen now lists Airtel Money / TNM Mpamba / Bank Transfer / Cash for MWK (screenshot below)

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

  • New Features
    • Added support for MWK currency with payment options including Airtel Money, TNM Mpamba, Bank Transfer, and Cash.

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.
@coderabbitai

coderabbitai Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 17613a89-b2e8-4148-a312-e3401e633456

📥 Commits

Reviewing files that changed from the base of the PR and between 5153994 and 64a79a1.

📒 Files selected for processing (1)
  • assets/data/payment_methods.json

Walkthrough

A single line is added to assets/data/payment_methods.json, mapping the MWK (Malawian Kwacha) currency key to the list ["Airtel Money", "TNM Mpamba", "Bank Transfer", "Cash"]. No existing entries are modified.

Changes

MWK Payment Methods Entry

Layer / File(s) Summary
MWK currency mapping
assets/data/payment_methods.json
Adds MWK key with ["Airtel Money", "TNM Mpamba", "Bank Transfer", "Cash"] to the payment methods map.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • MostroP2P/mobile#455: Also modifies assets/data/payment_methods.json by adding and restructuring per-currency payment method mappings, the same file and pattern as this PR.

Suggested reviewers

  • grunch

Poem

🐇 A new currency hops into view,
MWK joins the JSON queue!
Airtel Money, TNM Mpamba too,
Bank Transfer and Cash make four—it's true.
One tiny line, a country's methods appear,
The rabbit cheers: Malawi is here! 🌍

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding MWK payment methods to the configuration file, which is the sole purpose of this PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@grunch grunch left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@grunch grunch merged commit 0c3676d into MostroP2P:main Jun 22, 2026
2 checks passed
AndreaDiazCorreia added a commit that referenced this pull request Jun 24, 2026
revert: roll back Riverpod 3.x migration (#613) and dependent PRs (#624, #625)
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