Sendgrid-nodejs: REST-API-Details sollten zum Senden von E-Mails nicht erforderlich sein

Erstellt am 16. Sept. 2016  ·  4Kommentare  ·  Quelle: sendgrid/sendgrid-nodejs

Problemzusammenfassung

Helper sollte nicht verlangen, dass sich der Benutzer um Methoden oder Pfade kümmert: Es sollte den Leuten helfen, E-Mails mit vernünftigen Standardeinstellungen zu senden, wie jeder andere API-Client auf npm.

Schritte zum Reproduzieren

  1. Sehen Sie sich sendgrid v2 an.
  2. Verwenden Sie sendgrid v4 und fragen Sie sich, warum der Code von helper.Email() (zumindest gemäß den Beispielen) Konzepte auf niedrigerer Ebene erfordert. Es ist mir egal, was Ihr Endpunkt ist, es sei denn, ich überschreibe ihn, deshalb verwende ich Ihre Bibliothek und keine Rohanforderungen. Habe vernünftige Voreinstellungen.
question
Quelle
picture mikemaccana
👍3

Hilfreichster Kommentar

es sollte Leuten helfen, E-Mails mit vernünftigen Standardeinstellungen zu senden, wie jeder andere API-Client auf npm

Leider verhält sich dieses Modul eher wie ein einfaches Hilfsset zum Konstruieren von sendgrid-Nutzlasten. Und es sollte verhindern, dass ein falsch formatiertes Json überhaupt auf die API trifft. Zumindest wäre eine ordnungsgemäße Dokumentation dem Erlernen der Mängel des Moduls durch Fehler im API-Antworttext vorzuziehen. Braucht auf jeden Fall mehr Politur rundum.

picture eizzo am 16. Sept. 2016
👍2

Alle 4 Kommentare

es sollte Leuten helfen, E-Mails mit vernünftigen Standardeinstellungen zu senden, wie jeder andere API-Client auf npm

Leider verhält sich dieses Modul eher wie ein einfaches Hilfsset zum Konstruieren von sendgrid-Nutzlasten. Und es sollte verhindern, dass ein falsch formatiertes Json überhaupt auf die API trifft. Zumindest wäre eine ordnungsgemäße Dokumentation dem Erlernen der Mängel des Moduls durch Fehler im API-Antworttext vorzuziehen. Braucht auf jeden Fall mehr Politur rundum.

eizzo picture eizzo am 16. Sept. 2016
👍2

Hallo @mikemaccana ,

Sie haben Recht und Änderungen folgen in Kürze, siehe: https://github.com/sendgrid/sendgrid-nodejs/issues/290

@eizzo ,

Ja, der Zweck dieser ersten Überarbeitung bestand darin, Low-Level-Support für die gesamte v3-Web-API zu erhalten, während die enge Kopplung der vorherigen Bibliothek mit dem vorherigen E-Mail-Sendeendpunkt aufgehoben wurde. Wir laden Sie ein, uns auf dieser Reise zu begleiten, bitte sehen Sie sich dies an, wenn Sie einen Moment Zeit haben: https://github.com/sendgrid/sendgrid-nodejs/issues/290

Ich freue mich auf Ihre beiden Rückmeldungen, während wir den Weg der vollständigen Implementierung dieser Bibliothek fortsetzen.

Danke für deine Unterstützung!

Ich schließe dieses Ticket; Sie können jedoch gerne neue Issues oder Pull-Requests mit spezifischen Implementierungsdetails oder -ideen eröffnen.

thinkingserious picture thinkingserious am 16. Sept. 2016

Kommt bald irgendeine Hilfe, um diesen Helfer-Wahnsinn zu beseitigen/zu umgehen?

honkskillet picture honkskillet am 18. März 2017

Hallo @honkskillet ,

Jawohl :)

Bitte folgen Sie dem Projekt Mail Helper Enhancement (v3 mail/send), wenn Sie folgen oder einen Beitrag leisten möchten. @adamreisnz wird dort die neue

Mit freundlichen Grüßen,

Elmer

thinkingserious picture thinkingserious am 18. März 2017
War diese Seite hilfreich?
0 / 5 - 0 Bewertungen

Verwandte Themen

amlcodes picture amlcodes  ·  4Kommentare
thidasapankaja picture thidasapankaja  ·  4Kommentare
Loriot-n picture Loriot-n  ·  4Kommentare
metalshan picture metalshan  ·  3Kommentare
murphman300 picture murphman300  ·  4Kommentare