Skip to content

Conversation

@Borisw123
Copy link
Contributor

Replace logo list with table layout; remove MTN from currently supported providers list

Copilot AI review requested due to automatic review settings October 14, 2025 19:46
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Refactor the README.md to improve the visual presentation of supported payment providers by replacing the inline logo list with a structured table layout, and update the provider status by removing MTN from the currently supported providers.

  • Converted inline logo images to an organized HTML table format with consistent sizing and labeling
  • Removed MTN from the "Currently Supported Providers" section and updated its status in the roadmap

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines 404 to +406
- [x] Airtel Money
- [x] MTN
- [ ] PayStack
Copy link

Copilot AI Oct 14, 2025

Choose a reason for hiding this comment

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

The provider status updates are inconsistent. MTN is marked as completed in the roadmap (line 405) but was removed from the currently supported providers table above. PayStack appears in the supported providers table (line 43) but is marked as incomplete in the roadmap (line 406). These statuses should be aligned for consistency.

Copilot uses AI. Check for mistakes.
Comment on lines +51 to +53
<td align="center">
</td>
<td align="center">
Copy link

Copilot AI Oct 14, 2025

Choose a reason for hiding this comment

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

Empty table cells should be removed or filled with content. Consider removing the empty row or adding placeholder content to maintain a clean table structure.

Suggested change
<td align="center">
</td>
<td align="center">
<td align="center">
<em>Coming soon</em>
</td>
<td align="center">
<em>Coming soon</em>

Copilot uses AI. Check for mistakes.
@Einswilli Einswilli merged commit 5d73ba5 into AllDotPy:master Oct 14, 2025
1 check passed
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