Skip to content
This repository was archived by the owner on Oct 29, 2020. It is now read-only.

SMS report back for #ImABoss#2768

Merged
jonuy merged 1 commit intoDoSomethingArchive:devfrom
jonuy:imaboss-sms
Jul 7, 2014
Merged

SMS report back for #ImABoss#2768
jonuy merged 1 commit intoDoSomethingArchive:devfrom
jonuy:imaboss-sms

Conversation

@jonuy
Copy link
Contributor

@jonuy jonuy commented Jul 3, 2014

Same deal as the report backs from before.

mData endpoint setup to POST to dosomething.org/dosomething_sms/imaboss-reportback. The only real thing to note is $activity->nid = 2401; which should be the #ImABoss node id.

Tested stepping through the SMS flow on my local by posting requests at the endpoint to mimic Mobile Commons requests.

@jonuy jonuy changed the title SMS report back from #ImABoss SMS report back for #ImABoss Jul 3, 2014
@aaronschachter
Copy link
Contributor

👍

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps move this below $activity = $workflow->newActivity('end'); to complete the $activity = $workflow entries before assigning the Id values?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These id values apply to the $activity assigned above ($activity = $workflow->newActivity('dosomething_sms_submit_reportback');). So it needs to happen before the $activity = $workflow->newActivity('end');

@deezone
Copy link
Contributor

deezone commented Jul 3, 2014

👍

jonuy added a commit that referenced this pull request Jul 7, 2014
SMS report back for #ImABoss
@jonuy jonuy merged commit 5dc985f into DoSomethingArchive:dev Jul 7, 2014
@jonuy jonuy deleted the imaboss-sms branch July 7, 2014 13:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants