-
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
chore(release): pull release/v1.55.0 into main #3057
Conversation
Snyk has created this PR to upgrade ua-parser-js from 1.0.35 to 1.0.37. See this package in npm: https://www.npmjs.com/package/ua-parser-js See this project in Snyk: https://app.snyk.io/org/sandeep-L8FvsjCG7mBBqonjSmN48c/project/4b0f037e-0dab-4719-aeee-b46d2ae82119?utm_source=github&utm_medium=referral&page=upgrade-pr
Bumps [actions/setup-go](https://github.com/actions/setup-go) from 4.1.0 to 5.0.0. - [Release notes](https://github.com/actions/setup-go/releases) - [Commits](actions/setup-go@v4.1.0...v5.0.0) --- updated-dependencies: - dependency-name: actions/setup-go dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
…3037) * fix: adding map for marketo known values and javascript known values * fix: reverting launch.json to original state * fix: editing test cases * fix: review comments addressed * fix: making error statement better * feat: reviw comments addressed
) * feat: initial commit * fix: initial commit * fix: adding unauthorized error for access token failure * fix: reviw comments addressed * feat: reviw comments addressed * fix: changing version of integrations lib * Apply suggestions from code review Co-authored-by: Sankeerth <[email protected]> * fix: review comments addressed * fix: removing unnecessary tag * fix: removing unnecessary log * fix: review comments addressed * fix: adding destination response * Update src/v0/util/facebookUtils/networkHandler.js Co-authored-by: Sankeerth <[email protected]> --------- Co-authored-by: Sankeerth <[email protected]>
* feat(intercom): upgrade intercom version from 1.4 to 2.10 * chore: code review changes * chore: code review changes * chore: code review changes * chore: added backward compatibility for updateLastRequestAt field * chore: intercom backward compatibility support * chore: pr conflicts resolved * chore: pr conflicts resolved * chore: added step to check value of apiVersion * chore: change default value of apiVersion to latest * fix: intercom router response * chore: code review changes * chore: code review changes
* feat: initial commit * refactor: added comments * featL add payload builder utilities * feat: add privacy settings and canonical names * refactor: created separate transform.js for record and conversion use case * feat: added getRevenue utility and unit testcases * refactor: handle record/track type and added testcases * fix: use map instead of forEach for async operations * docs: add comments * feat: review changes * refactor: moved transform.js code logic to rtWorkflow * feat: added generate exclusion list utility * refactor: error message and function variable name * refactor: import endpoint from config and added api links * docs: add comments in response handler * refactor: created separate validateConfig utility for both flows * feat: updated category id mapping * test: add generateExclusionList testcases
* feat: onboard new destination rakuten_advertising * chore: small fix * chore: onboard rakuten proxy * chore: onboard rakuten proxy on v1 * chore: address comments and minor fixes * chore: small fix * chore: small fix for sonar issues * chore: small fix for sonar issues+1 * chore: small fix for sonar issues+3 * chore: making name as a required field for every product * Update src/cdk/v2/destinations/rakuten/utils.js Co-authored-by: Dilip Kola <[email protected]> * chore: address comments --------- Co-authored-by: Dilip Kola <[email protected]>
Important Auto Review SkippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
Test report for this run is available at: https://test-integrations-dev.s3.amazonaws.com/integrations-test-reports/rudder-transformer/3057/test-report.html |
Test report for this run is available at: https://test-integrations-dev.s3.amazonaws.com/integrations-test-reports/rudder-transformer/3057/test-report.html |
fix: trade desk extract custom properties util
Test report for this run is available at: https://test-integrations-dev.s3.amazonaws.com/integrations-test-reports/rudder-transformer/3057/test-report.html |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3057 +/- ##
==========================================
- Coverage 87.20% 87.19% -0.01%
==========================================
Files 530 537 +7
Lines 29079 29226 +147
Branches 6982 6963 -19
==========================================
+ Hits 25357 25484 +127
- Misses 3376 3394 +18
- Partials 346 348 +2 ☔ View full report in Codecov by Sentry. |
Test report for this run is available at: https://test-integrations-dev.s3.amazonaws.com/integrations-test-reports/rudder-transformer/3057/test-report.html |
Test report for this run is available at: https://test-integrations-dev.s3.amazonaws.com/integrations-test-reports/rudder-transformer/3057/test-report.html |
Quality Gate failedFailed conditions 18.3% Duplication on New Code (required ≤ 15%) |
👑 An automated PR