github / github/email_reply_parser

Dutch end of email is not detected correctly

Aberta
#41 1 comentário 2 reações 0 responsáveis Ver no GitHub
Linguagem predominante
Ruby
Estrelas
708
Forks
156
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

This part of the gem: `if text =~ /^(?!On.*On\s.+?wrote:)(On\s(.+?)wrote:)$/nm` works correctly for non-Dutch emails, but this is not correctly parsed as the end of the email:

`Op zondag 31 mei 2015 heeft Name het volgende geschreven:`

It would be awesome if this could be added!

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Search the gem for the shown `On ... wrote:` regular expression and inspect the email parsing path around it. Add coverage for the Dutch `Op zondag ... heeft ... het volgende geschreven:` example, then verify that it is detected as the end of the email without breaking the existing non-Dutch behavior.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
ruby
Domínio
backend
Tipo de issue
Bug
Dificuldade
2/5
Tempo estimado
1-3 horas
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
45/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.