Skip to content

Commit

Permalink
Merge branch 'develop' of github.com:rudderlabs/rudder-transformer in…
Browse files Browse the repository at this point in the history
…to refactor.migrate-test-cases-to-new-format
  • Loading branch information
Sai Sankeerth committed Sep 29, 2023
2 parents 9b4cfc6 + 56c80f2 commit f7acaa6
Show file tree
Hide file tree
Showing 12 changed files with 35 additions and 34 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/create-hotfix-branch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
runs-on: ubuntu-latest

# Only allow these users to create new hotfix branch from 'main'
if: github.ref == 'refs/heads/main' && (github.actor == 'ItsSudip' || github.actor == 'krishna2020' || github.actor == 'saikumarrs' || github.actor == 'sandeepdsvs' || github.actor == 'shrouti1507') && (github.triggering_actor == 'ItsSudip' || github.triggering_actor == 'krishna2020' || github.triggering_actor == 'saikumarrs' || github.triggering_actor == 'sandeepdsvs' || github.triggering_actor == 'shrouti1507')
if: github.ref == 'refs/heads/main' && (github.actor == 'ItsSudip' || github.actor == 'krishna2020' || github.actor == 'saikumarrs' || github.actor == 'sandeepdsvs' || github.actor == 'shrouti1507' || github.actor == 'anantjain45823') && (github.triggering_actor == 'ItsSudip' || github.triggering_actor == 'krishna2020' || github.triggering_actor == 'saikumarrs' || github.triggering_actor == 'sandeepdsvs' || github.triggering_actor == 'shrouti1507' || github.triggering_actor == 'anantjain45823')
steps:
- name: Create Branch
uses: peterjgrainger/[email protected]
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/draft-new-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ jobs:
runs-on: ubuntu-latest

# Only allow release stakeholders to initiate releases
if: (github.ref == 'refs/heads/develop' || startsWith(github.ref, 'refs/heads/hotfix/')) && (github.actor == 'ItsSudip' || github.actor == 'krishna2020' || github.actor == 'saikumarrs' || github.actor == 'sandeepdsvs' || github.actor == 'shrouti1507') && (github.triggering_actor == 'ItsSudip' || github.triggering_actor == 'krishna2020' || github.triggering_actor == 'saikumarrs' || github.triggering_actor == 'sandeepdsvs' || github.triggering_actor == 'shrouti1507')
if: (github.ref == 'refs/heads/develop' || startsWith(github.ref, 'refs/heads/hotfix/')) && (github.actor == 'ItsSudip' || github.actor == 'krishna2020' || github.actor == 'saikumarrs' || github.actor == 'sandeepdsvs' || github.actor == 'shrouti1507' || github.actor == 'anantjain45823') && (github.triggering_actor == 'ItsSudip' || github.triggering_actor == 'krishna2020' || github.triggering_actor == 'saikumarrs' || github.triggering_actor == 'sandeepdsvs' || github.triggering_actor == 'shrouti1507' || github.triggering_actor == 'anantjain45823')
steps:
- name: Checkout
uses: actions/[email protected]
Expand Down
8 changes: 8 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,14 @@

All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.

### [1.43.1](https://github.com/rudderlabs/rudder-transformer/compare/v1.43.0...v1.43.1) (2023-09-26)


### Bug Fixes

* **dm:** updated the stats definition for the tracking plan service ([eb2724f](https://github.com/rudderlabs/rudder-transformer/commit/eb2724f135d2c3c2a1fb8a88051cfb0f2720b5ed))
* **iterable:** event id mapping ([#2659](https://github.com/rudderlabs/rudder-transformer/issues/2659)) ([84083aa](https://github.com/rudderlabs/rudder-transformer/commit/84083aa671312e3934414b5534489fba87d9b29f))

## [1.43.0](https://github.com/rudderlabs/rudder-transformer/compare/v1.42.0...v1.43.0) (2023-09-25)


Expand Down
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "rudder-transformer",
"version": "1.43.0",
"version": "1.43.1",
"description": "",
"homepage": "https://github.com/rudderlabs/rudder-transformer#readme",
"bugs": {
Expand Down
16 changes: 2 additions & 14 deletions src/util/prometheus.js
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,7 @@ class Prometheus {
'destinationType',
'k8_namespace',
'workspaceId',
'transformationId',
'trackingPlanId',
],
},
{
Expand Down Expand Up @@ -586,7 +586,7 @@ class Prometheus {
'destinationType',
'k8_namespace',
'workspaceId',
'transformationId',
'trackingPlanId',
],
},
{
Expand Down Expand Up @@ -626,18 +626,6 @@ class Prometheus {
'k8_namespace',
],
},
{
name: 'user_transform_function_latency',
help: 'user_transform_function_latency',
type: 'histogram',
labelNames: [
'transformationVersionId',
'processSessions',
'sourceType',
'destinationType',
'k8_namespace',
],
},
{
name: 'source_transform_request_latency',
help: 'source_transform_request_latency',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,9 @@ const trackResponseBuilder = (message, metadata, destination) => {

const responseList = [];
if (!eventsToConversionsNamesMapping[event] || !eventsToOfflineConversionsTypeMapping[event]) {
throw new ConfigurationError(`Event name '${event}' is not valid`);
throw new ConfigurationError(
`Event name '${event}' is not present in the mapping provided in the dashboard.`,
);
}
const conversionTypes = Array.from(eventsToOfflineConversionsTypeMapping[event]);
conversionTypes.forEach((conversionType) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,9 @@ const getConversionActionId = async (headers, params) => {
'response.0.results.0.conversionAction.resourceName',
);
if (!conversionAction) {
throw new AbortedError(`Unable to find conversionActionId for conversion:${params.event}`);
throw new AbortedError(
`Unable to find conversionActionId for conversion:${params.event}. Most probably the conversion name in Google dashboard and Rudderstack dashboard are not same.`,
);
}
return conversionAction;
});
Expand Down Expand Up @@ -172,7 +174,7 @@ const requestBuilder = (
const filteredLoginCustomerId = removeHyphens(loginCustomerId);
response.headers['login-customer-id'] = filteredLoginCustomerId;
} else {
throw new ConfigurationError(`loginCustomerId is required as subAccount is enabled`);
throw new ConfigurationError(`"Login Customer ID" is required as "Sub Account" is enabled`);
}
}
return response;
Expand Down
7 changes: 5 additions & 2 deletions src/v0/destinations/iterable/data/IterableTrackConfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,11 @@
},
{
"destKey": "id",
"sourceKeys": ["properties.id", "properties.event_id"],
"required": false
"sourceKeys": "properties.event_id",
"required": false,
"metadata": {
"type": "toString"
}
},
{
"destKey": "eventName",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,11 @@
},
{
"destKey": "id",
"sourceKeys": [
"properties.order_id",
"properties.orderId",
"properties.id",
"properties.event_id"
],
"required": false
"sourceKeys": ["properties.order_id", "properties.orderId", "properties.event_id"],
"required": false,
"metadata": {
"type": "toString"
}
},
{
"destKey": "createdAt",
Expand Down
4 changes: 2 additions & 2 deletions test/__tests__/data/google_adwords_offline_conversions.json
Original file line number Diff line number Diff line change
Expand Up @@ -1622,7 +1622,7 @@
}
},
"statusCode": 400,
"error": "Event name 'purchase' is not valid",
"error": "Event name 'purchase' is not present in the mapping provided in the dashboard.",
"statTags": {
"destination": "google_adwords_offline_conversions",
"stage": "transform",
Expand Down Expand Up @@ -2948,7 +2948,7 @@
"secret": null
},
"statusCode": 400,
"error": "loginCustomerId is required as subAccount is enabled",
"error": "\"Login Customer ID\" is required as \"Sub Account\" is enabled",
"statTags": {
"destination": "google_adwords_offline_conversions",
"stage": "transform",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -472,7 +472,7 @@
],
"batched": false,
"statusCode": 400,
"error": "Event name 'order completed' is not valid",
"error": "Event name 'order completed' is not present in the mapping provided in the dashboard.",
"statTags": { "errorCategory": "dataValidation", "errorType": "configuration" },
"destination": {
"Config": {
Expand Down

0 comments on commit f7acaa6

Please sign in to comment.