papercups-io/papercups

Better phone number filtering for twillio and SMS in integrations page

Aperta

#816 aperta il 5 mag 2021

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (1 assegnatario)Elixir (566 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (6032 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Problem Right now if you enter a phone number without the area code or have the wrong formatting in phone number section it will fail silently and not work. We should make sure that there is a country code and that it can work regardless of the text formatting

Solution It should validate that it has a country code and we shouldn't care about formatting

Guida contributor

Direzione di ricerca
Indaga la gestione corrente dell'input del numero di telefono nella pagina delle integrazioni, probabilmente in un template Phoenix o LiveView. Cerca la logica di validazione nelle directory lib/ o web/. Considera l'uso di una libreria come ex phone number o l'implementazione di una semplice regex per la presenza del prefisso internazionale. Controlla i commenti per eventuali discussioni esistenti sull'approccio preferito. Il problema specifica di ignorare la formattazione, quindi concentrati sulla rimozione dei caratteri non numerici e sull'assicurare una lunghezza che implichi un prefisso internazionale (es. >= 10 cifre).
Tech stack
elixir
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65