Skip to content

Conversation

ntombing
Copy link
Contributor

Hey,

I've added a list of countries that allow and block card payments.

The initial request asked for both the allowed and blocked lists. However, a follow-up comment requested that I display only the blocked countries.

I included both lists for the preview, but based on feedback from all stakeholders, I will remove the allowed list if needed.

@ntombing ntombing requested review from max-koro and racene-swan July 24, 2025 09:44
Copy link

linear bot commented Jul 24, 2025

Copy link

netlify bot commented Jul 24, 2025

Deploy Preview for incomparable-tiramisu-91a96a ready!

Name Link
🔨 Latest commit a170521
🔍 Latest deploy log https://app.netlify.com/projects/incomparable-tiramisu-91a96a/deploys/68a70cd8b1e5f400092ecdf2
😎 Deploy Preview https://deploy-preview-358--incomparable-tiramisu-91a96a.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ntombing
Copy link
Contributor Author

Hey,

I've added a list of countries that allow and block card payments.

The initial request asked for both the allowed and blocked lists. However, a follow-up comment requested that I display only the blocked countries.

I included both lists for the preview, but based on feedback from all stakeholders, I will remove the allowed list if needed.

I opted to keep both the allowed and blocked country lists. Some partners may only be interested in scanning blocked countries vs. verifying that their country is approved. With this new tab structure, it's easier to switch between the two.

@max-koro max-koro force-pushed the DOC-509/denied-allowed-countries branch from 1210adb to 6143073 Compare August 21, 2025 09:15
Copy link
Contributor

@max-koro max-koro left a comment

Choose a reason for hiding this comment

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

Great work! Just a couple minor suggestions :)

@ntombing ntombing merged commit 28a8752 into main Aug 21, 2025
4 checks 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