hiro/Vulcan
1
0
Fork 0
mirror of https://github.com/vale981/Vulcan synced 2025-03-06 10:01:40 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #1651 from Enliven-se/vulcan-newsletter-mailchimp-error-handling

Browse source 
use correct code for Mailchimp "already subscribed" state
This commit is contained in:
Sacha Greif 2017-06-19 08:39:32 +09:00 • committed by GitHub
commit 5f95749f17
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/vulcan-newsletter/lib/server/integrations/mailchimp.js
View file

@ -47,7 +47,7 @@ if (settings) {
return {result: 'subscribed', ...subscribe}; return {result: 'subscribed', ...subscribe};
} catch (error) { } catch (error) {
// if the email is already in the Mailchimp list, no need to throw an error // if the email is already in the Mailchimp list, no need to throw an error
if (error.message === "214") { if (error.code === 214) {
return {result: 'already-subscribed'}; return {result: 'already-subscribed'};
} }
throw new Error("subscription-failed", error.message); throw new Error("subscription-failed", error.message);