Handle invalid unsubscriptions #7690

BKinahan · Mar 21, 2016

Add logic to handle requests to server for e-mail unsubscribe URLs when a matching user is not found.

Redirects to /map and informs the user to update e-mail settings from their profile.

Tested locally for each link, for both valid and invalid e-mail addresses, and passes npm test.

Closes #7686.

virus79 · Mar 21, 2016

Removed spam @sludge256

BerkeleyTrue · Mar 21, 2016

This else is unnecessary. You can remove it and outdent the content. Same goes for the code below.

camperbot · Mar 21, 2016

@BKinahan updated the pull request.

BerkeleyTrue · Mar 21, 2016

LGTM

BKinahan Handle invalid unsubscriptions

9504a5c

              });
-    -        return res.redirect('/unsubscribed');
-    -      });
+    +        return res.redirect('/map');
+    +      } else {
+    +        return user.updateAttribute('sendMonthlyEmail', false, (err) => {

FreeCodeCamp/FreeCodeCamp