search

DevProgress / HillaryBNB

Home sharing app for Hillary Clinton supporters
Other
10 stars 13 forks source link

Email contacts enhancements #78

Closed nellshamrell closed 8 years ago

nellshamrell commented 8 years ago

Fixes #63

dsjoerg commented 8 years ago

@nellshamrell my vote would be to leave this PR unmerged until the rest of #63 is done, or when our current primary user has an immediately upcoming campaign that needs Instant Emails.

The downside of merging this now is that hosts/organizers trying the system right now will receive multiple emails about the same visit which makes the system seem unnecessarily spammy. However if we have an imminent organizing campaign that needs Instant Emails we would merge in this PR pronto and take the risk.

@NanKrafft sound good to you?

@nellshamrell I'm optimistic that finishing #63 will be easy compared to all the work you've already done here.

nellshamrell commented 8 years ago

Taking a look at finishing #63 today :)

nellshamrell commented 8 years ago

This is now a complete fix for #63

dsjoerg commented 8 years ago

Excellent, thanks @nellshamrell !