DoSomething / voting-app

☑ Voting app for DoSomething.org campaigns.
MIT License
6 stars 4 forks source link

Message broker payload updates #449

Closed DFurnes closed 9 years ago

DFurnes commented 9 years ago

Changes

Some lovely payload updates for the Message Broker. See #437. There are a few open questions that I posted in the issue, but putting this code up for preliminary review. :eyeglasses:

Example Payloads

Brittany is a user from the US who gave us her email, but no cell phone. We'll send her some emails then:

screen shot 2015-11-12 at 10 39 16 am

Wilhelmina is a user from the US who gave us her email and her cell phone. She therefore must love texting, so we'll send things to her phone:

screen shot 2015-11-12 at 10 40 43 am

For review: @angaither @DeeZone

deezone commented 9 years ago

@DFurnes In the case of example 2, if the app collects email and mobile please send both sets of info:



In either or as a combo also send the MERGE_VARS values.

DFurnes commented 9 years ago

@DeeZone Done! Also addressed the feedback you provided in the issue.

DFurnes commented 9 years ago

@DeeZone The latest round of changes is ready for re-review! :)

DFurnes commented 9 years ago

Updated example payloads:

Email

Emailerina has provided her first name, birthdate, & email, and is from the US:

screen shot 2015-11-13 at 11 23 29 am

Mobile

Mobilerina has provided her first name, birthdate, email, & mobile number, and is from the US:

screen shot 2015-11-13 at 11 22 43 am
deezone commented 9 years ago

:+1:

angaither commented 9 years ago

:+1: