Closed ahmed-gelemli closed 1 month ago
Hi @ahmed-gelemli I'll review both of your PRs, but it looks like both PR is created from the same branch, so the commits are the same and bleeding into each other. If you can, it would make the review easier if you changed the branch for each PR in your fork. Cheers.
Hi @kuasha420 I did create different branches for each issue. Each PR has its own changes so it, I hope, is easier to review now.
Overview
This pull request adds a new feature to the PhoneNumberInput component that allows users to specify and select a list of countries. This enhances the flexibility of the component by enabling developers to restrict the country selection to a predefined list.
Changes Made
Added a new optional prop onlyCountries?: string[] to the PhoneNumberInput component.
Updated the README.md to include documentation for the new onlyCountries prop.
Provided an example usage of the onlyCountries prop in the README.md:
Testing
Verified that the component only displays the specified countries in the country picker when the onlyCountries prop is provided.
Ensured that the component functions as expected when the onlyCountries prop is not provided, showing all available countries.
Documentation
Updated the README.md file to reflect the addition of the onlyCountries prop.
Impact
This update is backward compatible and does not affect existing functionality. It simply adds an optional feature for greater customization.