Sendgrid-nodejs: Les détails de l'API REST ne devraient pas avoir besoin d'être inclus pour l'envoi d'e-mails

Créé le 16 sept. 2016  ·  4Commentaires  ·  Source: sendgrid/sendgrid-nodejs

Résumé de la question

Helper ne devrait pas obliger l'utilisateur à se soucier des méthodes ou des chemins : il devrait aider les gens à envoyer des e-mails avec des valeurs par défaut raisonnables, comme tous les autres clients API sur npm.

Étapes pour reproduire

  1. Regardez sendgrid v2.
  2. Utilisez sendgrid v4 et demandez-vous pourquoi le code helper.Email() (au moins selon les exemples) nécessite des concepts de niveau inférieur. Je me fiche de votre point de terminaison, à moins que je ne le remplace, c'est pourquoi j'utilise votre bibliothèque plutôt que des requêtes brutes. Avoir des valeurs par défaut raisonnables.
question

Commentaire le plus utile

cela devrait aider les gens à envoyer des e-mails avec des valeurs par défaut sensibles, comme tous les autres clients API sur npm

Malheureusement, ce module se comporte plus comme un ensemble d'assistants de base pour la construction de charges utiles sendgrid. Et cela devrait empêcher le json mal formaté d'atteindre l'API en premier lieu. À tout le moins, une documentation appropriée serait préférable à l'apprentissage des lacunes du module par des erreurs dans le corps de la réponse de l'API. A certainement besoin de plus de polissage tout autour.

Tous les 4 commentaires

cela devrait aider les gens à envoyer des e-mails avec des valeurs par défaut sensibles, comme tous les autres clients API sur npm

Malheureusement, ce module se comporte plus comme un ensemble d'assistants de base pour la construction de charges utiles sendgrid. Et cela devrait empêcher le json mal formaté d'atteindre l'API en premier lieu. À tout le moins, une documentation appropriée serait préférable à l'apprentissage des lacunes du module par des erreurs dans le corps de la réponse de l'API. A certainement besoin de plus de polissage tout autour.

Bonjour @mikemaccana ,

Vous avez raison et des changements arrivent bientôt, veuillez consulter : https://github.com/sendgrid/sendgrid-nodejs/issues/290

@eizzo ,

Oui, le but de cette première réécriture était d'obtenir une prise en charge de bas niveau pour toutes les API Web v3, tout en supprimant le couplage étroit que la bibliothèque précédente avait sur le point de terminaison d'envoi de courrier précédent. Nous vous invitons à nous rejoindre dans ce voyage, veuillez jeter un œil à ceci lorsque vous aurez un moment : https://github.com/sendgrid/sendgrid-nodejs/issues/290

J'attends avec impatience vos deux commentaires alors que nous poursuivons la mise en œuvre complète de cette bibliothèque.

Merci pour votre aide!

Je ferme ce ticket ; cependant, n'hésitez pas à ouvrir de nouveaux problèmes ou des demandes d'extraction avec des détails ou des idées de mise en œuvre spécifiques.

De l'aide sera bientôt disponible pour supprimer/éviter cette folie des assistants ?

Salut @honkskilet ,

Oui :)

Veuillez suivre le projet Mail Helper Enhancement (v3 mail/send) si vous souhaitez suivre ou contribuer. @adamreisnz y proposera la nouvelle interface d'assistance pour les commentaires de la communauté.

Meilleures salutations,

Elmer

Cette page vous a été utile?
0 / 5 - 0 notes