Skip to content
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

fix: rudderlabs pull to team #74

Merged
merged 353 commits into from
Dec 18, 2023
Merged

Conversation

sumitsheokand
Copy link

ONCEHUB-75672: rudderlabs pull to team

JustinFernandes19 and others added 30 commits September 20, 2023 12:07
feat: [qa] oncehub-71928 : update rudder version to latest (team to master merge)
* fix: bqstream event ordering fix

* fix: review comments addressed

* fix: importing the common util function

* fix: putting both error and successful events logic back

* fix: test cases addition and code clean up

* fix: code clean up

* fix: resolving sonar error

* fix: removing network handler for bqstream

* fix: refactor process router dest
…ies (rudderlabs#2634)

fix(mp incremental properties): add check for undefined properties
…nation (rudderlabs#2652)

* feat: update google_cloud_function implementation as per custom destination flow

* fix: remove old test suit
…dderlabs#2656)

fix: fb_error message for invalid type of property value
* feat(iterable): user deletion support

* feat(iterable): user deletion improvements

* chore: added comments
chore: update release stakeholder
* chore: partial failure handling

* chore: code review changes
yashasvibajpai and others added 28 commits December 11, 2023 16:36
* chore: init onboard

* chore: add network handler

* chore: add test

* fix: adding correct mock with corrections & modify the status-code in test-case

Signed-off-by: Sai Sankeerth <[email protected]>

* chore: add dummy destination id

* chore: add dummy ws id

* chore: add tests for success and generic error case

---------

Signed-off-by: Sai Sankeerth <[email protected]>
Co-authored-by: Sai Sankeerth <[email protected]>
* fix(integrations): onboard sprig destination

* chore: code review changes

* chore: code review changes
When message doesn't contain event field there is no meaningful
process we can do so considering it as instrumenation error.
rudderlabs#2897)

Found logs for this error:
Failure occurred during custom attributes operations
TypeError: Cannot read properties of null (reading 'update')
…erlabs#2889)

mailjet code uses md5 method to hash email for using it as userId
when actual userId is not found in the event and when email doesn't
exists then md5 throwing an error so added check before calling
md5 method.
* fix: covert toString before toLowercase

* fix: throw error if not valid email
* feat: tiktok_ads: add support for custom events

* chore: added doc link
* fix: send response object captured after API call to active_campaign error handler

Signed-off-by: Sai Sankeerth <[email protected]>

* fix: add test-cases for 5xx, 4xx failures

Signed-off-by: Sai Sankeerth <[email protected]>

---------

Signed-off-by: Sai Sankeerth <[email protected]>
Co-authored-by: Sai Sankeerth <[email protected]>
…labs#2899)

* dropped support of useClones for trackingplans cache

* updated the changelog
…labs#2904)

* chore: add lowercase check to MSL for event type audiencelist

* chore: fix optional chaining

Co-authored-by: Utsab Chowdhury <[email protected]>

* chore: update test

---------

Co-authored-by: Utsab Chowdhury <[email protected]>
@SoniRani SoniRani merged commit f3a4146 into team/technocrats Dec 18, 2023
2 of 8 checks passed
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.