Sendgrid-nodejs: कॉलबैक फ़ंक्शन हस्ताक्षर नोड सम्मेलनों का पालन नहीं करता है

को निर्मित 15 जून 2016 · 25टिप्पणियाँ · स्रोत: sendgrid/sendgrid-nodejs

यह शायद एक निर्णय आप चीजों को एक निश्चित तरीके से करने के लिए किया जाता है, लेकिन यह इस पुस्तकालय की 3.0.4 संस्करण के साथ लगता है, कॉलबैक करने के लिए पारित कर दिया sg.API मानक नोड कॉलबैक के समारोह हस्ताक्षर सम्मेलन का पालन नहीं करता function (error, data) । इसके बजाय, एपीआई प्रतिक्रिया को हमेशा पहली वस्तु के रूप में लौटाया जाता है।

इससे यह निर्धारित करना कठिन हो जाता है कि क्या कुछ गलत हुआ था, क्योंकि अब हम कच्चे एपीआई प्रतिक्रिया को डिक्रिप्ट करने के लिए जिम्मेदार हैं और मैन्युअल रूप से प्रतिक्रिया हेडर और स्थिति कोड का विश्लेषण करते हैं।

मुझे नहीं लगता कि यह उपयोगकर्ता भूमि की जिम्मेदारी होनी चाहिए, लेकिन पुस्तकालय की ही। क्या होगा अगर स्थिति कोड बदल जाते हैं? क्या होगा अगर प्रतिक्रिया स्वरूप बदलता है? क्या होगा यदि कोई त्रुटि होती है और लौटाया गया डेटा प्रारूप अलग, या अशक्त या अपरिभाषित हो जाता है?

यह पता करने के लिए बहुत कम से कम किया जा सकता है कि लाइब्रेरी के लिए यह निर्धारित करना है कि कोई त्रुटि हुई है, और यदि हाँ, पहले पैरामीटर को एक त्रुटि ऑब्जेक्ट के साथ पॉप्युलेट करें (इसमें प्रतिक्रिया संलग्न हो सकती है)। यदि कोई त्रुटि नहीं होती है, तो पहले पैरामीटर के रूप में null वापस करें और दूसरे पैरामीटर के रूप में प्रतिक्रिया ऑब्जेक्ट।

help wanted community enhancement

स्रोत

adamreisnz

👍5

सबसे उपयोगी टिप्पणी

मेरा वर्तमान समाधान सफलता के लिए स्थिति कोड की जांच करना है, जैसे:

function sendMail(mail) {
  return new Promise((resolve, reject) => {

    //Build request
    let request = sg.emptyRequest();
    request.method = 'POST';
    request.path = '/v3/mail/send';
    request.body = mail.toJSON();

    //Send request
    sg.API(request, response => {
      if (response && response.statusCode &&
        response.statusCode >= 200 && response.statusCode <= 299) {
        resolve(response);
      }
      reject(new SendMailError(
        'Sendgrid response error ' + response.statusCode
      ));
    });
  });
}

जैसा कि आप देख सकते हैं कि यह काफी क्रियात्मक है और इसके लिए बहुत सारे बॉयलरप्लेट की आवश्यकता होती है, बस एक ईमेल भेजने और जांचने के लिए कि यह काम किया है या नहीं।

प्रस्तावित एपीआई कुछ इस तरह होगा:

function sendMail(mail) {
  return new Promise((resolve, reject) => {

    //Build request
    let request = sg.emptyRequest();
    request.method = 'POST';
    request.path = '/v3/mail/send';
    request.body = mail.toJSON();

    //Send request
    sg.API(request, (error, response) => {
      if (error {
        reject(new SendMailError(error);
      }
      resolve(response);
    });
  });
}

या बेहतर अभी तक, वादे के साथ लौटे:

function sendMail(mail) {

  //Build request
  let request = sg.emptyRequest();
  request.method = 'POST';
  request.path = '/v3/mail/send';
  request.body = mail.toJSON();

  //Send request
  return sg.API(request);
}

और, आदर्श रूप से, हम एक समर्पित मेल भेजने वाले सहायक की वापसी देखेंगे, जो बैकमेल हेल्पर और बॉयलरप्लेट की आवश्यकता को समाप्त करते हुए, पृष्ठभूमि में हमारे लिए उचित अनुरोध बनाएगा:

sg.sendMail(mail);

अगर यह कुछ ऐसा है जिसे आप आगे बढ़ाने में रुचि रखते हैं तो मैं इसके लिए कुछ पीआर बनाने पर ध्यान दूंगा।

adamreisnz 15 जून 2016

👍4

सभी 25 टिप्पणियाँ

मेरा वर्तमान समाधान सफलता के लिए स्थिति कोड की जांच करना है, जैसे:

function sendMail(mail) {
  return new Promise((resolve, reject) => {

    //Build request
    let request = sg.emptyRequest();
    request.method = 'POST';
    request.path = '/v3/mail/send';
    request.body = mail.toJSON();

    //Send request
    sg.API(request, response => {
      if (response && response.statusCode &&
        response.statusCode >= 200 && response.statusCode <= 299) {
        resolve(response);
      }
      reject(new SendMailError(
        'Sendgrid response error ' + response.statusCode
      ));
    });
  });
}

प्रस्तावित एपीआई कुछ इस तरह होगा:

function sendMail(mail) {
  return new Promise((resolve, reject) => {

    //Build request
    let request = sg.emptyRequest();
    request.method = 'POST';
    request.path = '/v3/mail/send';
    request.body = mail.toJSON();

    //Send request
    sg.API(request, (error, response) => {
      if (error {
        reject(new SendMailError(error);
      }
      resolve(response);
    });
  });
}

या बेहतर अभी तक, वादे के साथ लौटे:

function sendMail(mail) {

  //Build request
  let request = sg.emptyRequest();
  request.method = 'POST';
  request.path = '/v3/mail/send';
  request.body = mail.toJSON();

  //Send request
  return sg.API(request);
}

sg.sendMail(mail);

adamreisnz 15 जून 2016

👍4

धन्यवाद @adambuczynski ,

आम तौर पर, हम एरर हैंडलिंग को अपडेट करने की योजना बना रहे हैं, लेकिन हमने इसे अभी तक समाप्त नहीं किया है। उस समय के आने पर हम इस प्रतिक्रिया पर विचार करेंगे।

यदि आप एक पुल अनुरोध प्रदान करना चाहते हैं, तो हमें एक हस्ताक्षरित सीएलए चाहिए: https://github.com/sendgrid/sendgrid-nodejs/blob/master/CONTRIBUTING.md#cla

हमने आपके द्वारा प्रस्तावित प्रकारों में वृद्धि के लिए एक सहायक संरचना निर्धारित की है: https://github.com/sendgrid/sendgrid-nodejs/tree/master/lib/helpers/mail। आप या तो उस एक को संशोधित करने में मदद कर सकते हैं, या हेल्पर्स डायरेक्टरी में अपना खुद का बना सकते हैं।

आपके समर्थन के लिए धन्यवाद!

thinkingserious 15 जून 2016

हां मैंने हेल्पर को मेल के लिए देखा है। क्या आप उस वस्तु पर सहायक भेजने वाला मेल बनाना चाहते हैं?

adamreisnz 15 जून 2016

👍1

ये मुझे अच्छा लग रहा है :)

thinkingserious 15 जून 2016

@Adambuczynski किए गए सुझाव हाजिर हैं। मैंने अभी v3 एपीआई को अपडेट किया और बहुत निराश था कि मैं अब एक साधारण सहायक फ़ंक्शन का उपयोग नहीं कर सकता और इसके बजाय बॉयलर प्लेट कोड का एक गुच्छा लिखना होगा। मैं अब इसे बदसूरत त्रुटि कोड लिखने के बिना ब्लूबर्ड वादा में लपेट नहीं सकता।

beldenge 21 जून 2016

👍1

विस्मयकारी, @ शिष्टाचार के लिए धन्यवाद!

कृपया प्रगति के लिए इस मुद्दे का पालन करें। यदि @adambuczynski एक पुल अनुरोध बनाता है, तो मैं इसे यहाँ पोस्ट करूँगा; अन्यथा, हम इसे अगले लाइब्रेरी अपडेट में से एक के लिए अपने रोडमैप में जोड़ देंगे।

thinkingserious 21 जून 2016

@thinkingsouble मैंने अभी आपको हस्ताक्षर किए हुए CA को मेल किया है, मुझे उम्मीद है कि अगले हफ्ते कुछ समय लगेगा।

adamreisnz 21 जून 2016

👍1

@adambuczynski हम इसे प्राप्त किया है और प्रतिक्रिया के लिए धन्यवाद!

thinkingserious 21 जून 2016

👍1

हाय दोस्तों, इस बारे में नहीं भूल गया है, लेकिन मैं कई अन्य परियोजनाओं के साथ थोड़ा व्यस्त रहा हूँ। अभी भी कुछ चरण में इस पर गौर करने की उम्मीद :)

adamreisnz 3 जुल॰ 2016

👍1

@thinkingsouble मैंने आपके स्रोत कोड पर एक नज़र

क्या आप इसके साथ ठीक हैं? मुझे नहीं लगता कि मैं कोड के साथ काम कर पाऊंगा, अन्यथा, यह देखने के लिए मेरे मस्तिष्क को चोट पहुँचाता है :)

पुनश्च आप क्यों नहीं उपयोग करते हैं लिंटर :)

adamreisnz 26 जुल॰ 2016

@adambuczynski ,

हम इस लाइनर का उपयोग करते हैं: http://eslint.org मानक शैली गाइड के साथ।

आप कौन सा इस्तेमाल करते हैं? यदि हम सिंक कर सकते हैं और उसी लिंटर का उपयोग कर सकते हैं जो अद्भुत होगा।

thinkingserious 26 जुल॰ 2016

मैं ESLiint का भी उपयोग करता हूं, लेकिन ऐसा नहीं लगता कि आप बहुत सारे नियम लागू करते हैं क्योंकि कोड में अभी भी बहुत सारी विसंगतियां हैं।

यह वह कॉन्फ़िगरेशन है जिसका मैं उपयोग करता हूं: https://gist.github.com/adambuczynski/1fa24bcfc5d17b8d26e4c39ffca7560e#file -eslintrc-node-yaml

मुझे लगता है कि यह स्थिरता और सर्वोत्तम प्रथाओं के बीच एक अच्छा संतुलन प्रदान करता है, फिर भी बहुत कष्टप्रद होने के बिना।

adamreisnz 26 जुल॰ 2016

आपके प्रोजेक्ट में .eslintrc.yaml कॉन्फिग फ़ाइल नहीं है। क्या आप अपने पसंदीदा नियमों के साथ एक जोड़ / बना सकते हैं ताकि मैं भी इसका उपयोग कर सकूं?

adamreisnz 26 जुल॰ 2016

👍1

@adambuczynski ,

मैं तुम्हारा उपयोग करने के लिए खुश हूँ, यह ठोस लग रहा है :)

कृपया इसे अपनी प्रतिबद्धता के हिस्से के रूप में जोड़ें।

thinkingserious 26 जुल॰ 2016

👍1

@thinkingserious यह, उदाहरण के लिए चेतावनी है जब आप का उपयोग ES6 के लिए सेटअप सुविधाओं हालांकि है var के बजाय let और ES6 पार्सर का उपयोग कर।

क्या यह ठीक है, या आप केवल पीछे की संगतता के लिए ES5 का उपयोग करना पसंद करते हैं?

adamreisnz 27 जुल॰ 2016

हमें Node.js संस्करणों का समर्थन करने की आवश्यकता है: "0.10", "0.12", "iojs", "4"

यदि हम उन संस्करणों के लिए समर्थन नहीं तोड़ते हैं तो मैं अधिक आधुनिक दृष्टिकोण के साथ ठीक हूं।

thinkingserious 27 जुल॰ 2016

ठीक है, मैंने संस्करण की और अभी तय की गई कोड शैली मुद्दों को रखा है। इस मुद्दे से अलग उसके लिए एक पीआर बनाया।

adamreisnz 27 जुल॰ 2016

🎉1

@ वांछनीय

कई समाधान, मुझे पता है जो (या संयोजन) आप पसंद करते हैं:

एक निर्भरता के रूप में bluebird वादों का उपयोग करें
जांचें कि क्या वादे समर्थित हैं या नहीं, अगर वे नहीं हैं तो एक त्रुटि फेंक दें, लेकिन यदि लोग उनका उपयोग करने का प्रयास करते हैं
उपयोगकर्ताओं को वादे के कार्यान्वयन को निर्धारित करने की अनुमति दें, जो वे केवल sendgrid.Promise जो भी मूल्य निर्धारित करना चाहते हैं।

संभवतः 2 और 3 का संयोजन सबसे सरल और सबसे अधिक लचीला होगा।

adamreisnz 27 जुल॰ 2016

👍1

माना।

मुझे यह विकल्प पसंद है: "जांचें कि क्या वादे समर्थित हैं या नहीं, एक त्रुटि फेंक दें यदि वे नहीं हैं लेकिन अगर लोग उनका उपयोग करने का प्रयास करते हैं", लेकिन उपयोगकर्ताओं को वे कार्यान्वयन को निर्दिष्ट करने की अनुमति भी देते हैं जो वे चाहते हैं।

thinkingserious 27 जुल॰ 2016

👍1

@ धार्मिक

adamreisnz 27 जुल॰ 2016

@adambuczynski आप एक ही पीआर में रख सकते हैं, धन्यवाद!

thinkingserious 27 जुल॰ 2016

# 261 देखें

adamreisnz 27 जुल॰ 2016

👍1

@beldenge क्या आप किसी +1 को PR https://github.com/sendgrid/sendgrid-nodejs/pull/261 से जोड़ना पसंद करेंगे, ताकि Sendgrid टीम इसे स्थानांतरित कर सके और इसे मर्ज करने पर विचार कर सके? धन्यवाद!

adamreisnz 27 जुल॰ 2016

@adambuczynski मैं अब +1

thinkingserious 27 जुल॰ 2016

Https://github.com/sendgrid/sendgrid-nodejs/releases/tag/v4.0.0 के साथ फिक्स्ड

thinkingserious 2 अग॰ 2016

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Sendgrid-nodejs: कॉलबैक फ़ंक्शन हस्ताक्षर नोड सम्मेलनों का पालन नहीं करता है

सबसे उपयोगी टिप्पणी

सभी 25 टिप्पणियाँ

संबंधित मुद्दों