-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LAUD-302 - Comprehensive Multisplit #334
base: staging
Are you sure you want to change the base?
Conversation
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Hey @abheek9 Please check out these notes NotesDuring the testing, I found it unnecessary to manually import CSV files since the process involves numerous steps and needs to be performed in multiple tests repeatedly. Instead, I created a dedicated test specifically for the import CSV functionality. In this multi-split test, I addressed the import CSV functionality by directly issuing the network requests it involves. However, we can still include the entire Regarding the ticket description, it states that we should issue network requests to the test_multisplit_conditions.mp4 |
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
@gitstart-laudspeaker this test does not look correct, right now each branch is using the same webhook, but in the instructions, each branch uses a different webhook. This is important because we need to know which branch is triggered from a specific event |
Hey @abheek9 The webhooks are mentioned in the video description of the video, but it only mentions that the webhook can be our events endpoint. If you look closely at the webhooks, you notice that 3 branches share a webhook, and 2 also share a webhook. |
Hey @abheek9 Earlier I had asked for clarification on how the events and the webhooks are connected, now it makes sense. So, if we use the events endpoint in the webhooks, then the multisplit branches can each trigger a different event, we can then check the triggered events to confirm that the multisplit actually triggers the webhooks |
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Hey @abheek9 multi-split.mp4
However, I cannot see the events in the event-tracker meaning the API calls are not actually made, at what point are the users enrolled in the journeys(I also see no users in the journey) so that the web-hooks are triggered. I also created the journey manually as a normal user would and still don't see the events. Perhaps you can provide some insight into how the various branches are triggered. events.mp4 |
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Co-authored-by: v1b3m <vibenjamin6@gmail.com>
Description
Comprehensive multisplit
Refs
#302
Demo
Multisplit
test_multisplit.mp4
uploadCSV
uploadCSV.mp4
Fixes #302