Automattic / Automattic/php-phone

Iso3166 class may need to be renamed to fit its purpose better.

Open
#11 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
PHP
Stars
10
Forks
9
PR merge metrics
No merged PRs in 30d

Description

The ISO3166 standard actually have nothing to do with phone numbers. It is about country identifiers in two letters (alpha-2), three letters (alpha-3), three digits (numeric-3), and country names. So, **possible mobile number prefixes** and **possible phone number lengths** are not part of ISO3166. It actually took me a while to figure this out at the beginning.

In case of any future confusion, we may need a better name for it.

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the Iso3166 class and review how its name relates to the country identifiers, phone-number prefixes, and phone-number lengths it exposes. Check the repository for references to the class, then determine and document an appropriate scope for renaming it; the issue does not specify a proposed replacement name or tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.