papercups-io/papercups

Better phone number filtering for twillio and SMS in integrations page

Aberta

#816 aberto em 5 de mai. de 2021

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (1 responsável)Elixir (566 forks)batch import
enhancementgood first issue

Métricas do repositório

Stars
 (6.032 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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

Guia do colaborador

Direção de pesquisa
Investigue o tratamento atual da entrada de número de telefone na página de integrações, provavelmente em um template Phoenix ou LiveView. Procure a lógica de validação nos diretórios lib/ ou web/. Considere usar uma biblioteca como ex phone number ou implementar uma regex simples para verificar a presença do código do país. Verifique os comentários para quaisquer discussões existentes sobre a abordagem preferida. A issue especifica ignorar a formatação, então foque em remover caracteres não numéricos e garantir um comprimento que implique um código do país (ex.: >= 10 dígitos).
Pilha de tecnologia
elixir
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65