Generalize M-Pesa flow into multi-provider mobile money support #122

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Peter-John-paystack merged 2 commits into main from feature/Generalize-into-multi-provider-mobile-money-support

May 20, 2026

Merged

Generalize M-Pesa flow into multi-provider mobile money support #122

Updating tests to reflect the correct keys

SonarQubeCloud / SonarCloud Code Analysis failed May 20, 2026 in 16s

Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 89 in Sources/PaystackUI/Charge/MobileMoney/Views/MobileMoneyProcessingView.swift

sonarqubecloud / SonarCloud Code Analysis

This operation returns the same value whether the condition is "true" or "false".

See more on https://sonarcloud.io/project/issues?id=PaystackHQ_paystack-sdk-ios&issues=AZ5F1mk_Pw7KcwPP0-ji&open=AZ5F1mk_Pw7KcwPP0-ji&pullRequest=122

Check failure on line 108 in Sources/PaystackUI/Charge/MobileMoney/Views/MobileMoneyProcessingView.swift

sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this closure is empty, or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=PaystackHQ_paystack-sdk-ios&issues=AZ5F1mk_Pw7KcwPP0-jj&open=AZ5F1mk_Pw7KcwPP0-jj&pullRequest=122

View more details on SonarQubeCloud