Skip to content

Pass segmentId to automation processing#1055

Merged
sausage-todd merged 2 commits intomainfrom
automations-segments-fix
Jun 28, 2023
Merged

Pass segmentId to automation processing#1055
sausage-todd merged 2 commits intomainfrom
automations-segments-fix

Conversation

@sausage-todd
Copy link
Copy Markdown
Contributor

@sausage-todd sausage-todd commented Jun 28, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at b7c7ad4

Added support for segment-based automations for new activity and new member triggers. Modified the message types, SQS functions, and workers to pass and use the segmentId property. Updated the memberService to send the segmentId when creating or updating a member.

🤖 Generated by Copilot at b7c7ad4

To automate actions with ease
We added segmentId keys
To the SQS messages
And the worker processes
That handle the members and activities

Why

How

🤖 Generated by Copilot at b7c7ad4

  • Add segmentId parameter to new activity and new member automation workers and messages (link, link, link, link, link, link, link, link, link)
  • Pass segmentId from options to sendNewMemberNodeSQSMessage function in memberService.ts (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@sausage-todd sausage-todd force-pushed the automations-segments-fix branch 3 times, most recently from 69a1b45 to c42d5c9 Compare June 28, 2023 14:31
@sausage-todd sausage-todd force-pushed the automations-segments-fix branch from c42d5c9 to 77fb47a Compare June 28, 2023 14:35
@sausage-todd sausage-todd force-pushed the automations-segments-fix branch from 77fb47a to 907368a Compare June 28, 2023 15:08
@sausage-todd sausage-todd merged commit 88d0049 into main Jun 28, 2023
@sausage-todd sausage-todd deleted the automations-segments-fix branch June 28, 2023 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant