github / github/email_reply_parser

Dutch end of email is not detected correctly

Offen
#41 1 Kommentar 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Ruby
Sterne
708
Forks
156
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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!

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
ruby
Bereich
backend
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.