chore(release): pull hotfix-release/v1.126.1 into main#5044
Conversation
## What are the changes introduced in this PR? Added one check for validating if the context is defined for an identify call or not. ## What is the related Linear task? Resolves INT-6054 ## Please explain the objectives of your changes below Put down any required details on the broader aspect of your changes. If there are any dependent changes, **mandatorily** mention them here ### Any changes to existing capabilities/behaviour, mention the reason & what are the changes ? N/A ### Any new dependencies introduced with this change? N/A ### Any new generic utility introduced or modified. Please explain the changes. N/A ### Any technical or performance related pointers to consider with the change? N/A @coderabbitai review <hr> ### Developer checklist - [ ] My code follows the style guidelines of this project - [ ] **No breaking changes are being introduced.** - [ ] All related docs linked with the PR? - [ ] All changes manually tested? - [ ] Any documentation changes needed with this change? - [ ] Is the PR limited to 10 file changes? - [ ] Is the PR limited to one linear task? - [ ] Are relevant unit and component test-cases added in **new readability format**? ### Reviewer checklist - [ ] Is the type of change in the PR title appropriate as per the changes? - [ ] Verified that there are no credentials or confidential data exposed with the changes.
#5038) ## What are the changes introduced in this PR? We are getting some unexpected errors, so adding a logger to verify why those error might happen. Along with that, we are considering all 2xx as a success scenario as we can't consider them as failures. ## What is the related Linear task? Resolves INT-6069 ## Please explain the objectives of your changes below Put down any required details on the broader aspect of your changes. If there are any dependent changes, **mandatorily** mention them here ### Any changes to existing capabilities/behaviour, mention the reason & what are the changes ? N/A ### Any new dependencies introduced with this change? N/A ### Any new generic utility introduced or modified. Please explain the changes. N/A ### Any technical or performance related pointers to consider with the change? N/A @coderabbitai review <hr> ### Developer checklist - [ ] My code follows the style guidelines of this project - [ ] **No breaking changes are being introduced.** - [ ] All related docs linked with the PR? - [ ] All changes manually tested? - [ ] Any documentation changes needed with this change? - [ ] Is the PR limited to 10 file changes? - [ ] Is the PR limited to one linear task? - [ ] Are relevant unit and component test-cases added in **new readability format**? ### Reviewer checklist - [ ] Is the type of change in the PR title appropriate as per the changes? - [ ] Verified that there are no credentials or confidential data exposed with the changes.
🔒 Scanned for secrets using gitleaks 8.28.0 ## What are the changes introduced in this PR? We are considering email form data fields as well. ## What is the related Linear task? Resolves INT- ## Please explain the objectives of your changes below Put down any required details on the broader aspect of your changes. If there are any dependent changes, **mandatorily** mention them here ### Any changes to existing capabilities/behaviour, mention the reason & what are the changes ? N/A ### Any new dependencies introduced with this change? N/A ### Any new generic utility introduced or modified. Please explain the changes. N/A ### Any technical or performance related pointers to consider with the change? N/A @coderabbitai review <hr> ### Developer checklist - [ ] My code follows the style guidelines of this project - [ ] **No breaking changes are being introduced.** - [ ] All related docs linked with the PR? - [ ] All changes manually tested? - [ ] Any documentation changes needed with this change? - [ ] Is the PR limited to 10 file changes? - [ ] Is the PR limited to one linear task? - [ ] Are relevant unit and component test-cases added in **new readability format**? ### Reviewer checklist - [ ] Is the type of change in the PR title appropriate as per the changes? - [ ] Verified that there are no credentials or confidential data exposed with the changes.
…5041) 🔒 Scanned for secrets using gitleaks 8.28.0 We are getting email inside `data.checkout` for checkout-related events like `checkout_started`, `checkout_address_info_submitted`, `payment_info_submitted`, etc. But we are not mapping them properly to `context.traits.email`, which is breaking the event flow in downstream destinations like Google Ads enhanced conversion. So, updated the logic to consider the mapping for checkout-related events. ## What is the related Linear task? Resolves INT-6075 ## Please explain the objectives of your changes below Put down any required details on the broader aspect of your changes. If there are any dependent changes, **mandatorily** mention them here ### Any changes to existing capabilities/behaviour, mention the reason & what are the changes ? N/A ### Any new dependencies introduced with this change? N/A ### Any new generic utility introduced or modified. Please explain the changes. N/A ### Any technical or performance related pointers to consider with the change? N/A @coderabbitai review <hr> ### Developer checklist - [ ] My code follows the style guidelines of this project - [ ] **No breaking changes are being introduced.** - [ ] All related docs linked with the PR? - [ ] All changes manually tested? - [ ] Any documentation changes needed with this change? - [ ] Is the PR limited to 10 file changes? - [ ] Is the PR limited to one linear task? - [ ] Are relevant unit and component test-cases added in **new readability format**? ### Reviewer checklist - [ ] Is the type of change in the PR title appropriate as per the changes? - [ ] Verified that there are no credentials or confidential data exposed with the changes.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Note
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5044 +/- ##
==========================================
- Coverage 92.32% 92.27% -0.05%
==========================================
Files 657 656 -1
Lines 36139 35915 -224
Branches 8531 8470 -61
==========================================
- Hits 33365 33142 -223
+ Misses 2557 2535 -22
- Partials 217 238 +21 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|



👑 An automated PR