Skip to content

Commit 741f525

Browse files
committed
Reorder dataLayer event and variable pushes
Moved the push of individual dataLayer events to occur after pushing all collected variables. This ensures that the 'pageJsonVariablesReady' event is pushed before any GTM-required events, aligning with expected dataLayer processing order.
1 parent 93d78c1 commit 741f525

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

packages/scripts/dist/data-layer.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -170,15 +170,15 @@ export class DataLayer {
170170
else {
171171
dataLayerEvents.push("EN_FASTFORMFILL_ALL_FAILURE");
172172
}
173-
// Push all collected events individually (GTM requirement)
174-
dataLayerEvents.forEach((event) => {
175-
this.dataLayer.push({ event });
176-
});
177173
// Push all collected variables at once
178174
if (Object.keys(dataLayerData).length > 0) {
179175
dataLayerData.event = "pageJsonVariablesReady";
180176
this.dataLayer.push(dataLayerData);
181177
}
178+
// Push all collected events individually (GTM requirement)
179+
dataLayerEvents.forEach((event) => {
180+
this.dataLayer.push({ event });
181+
});
182182
this.attachEventListeners();
183183
}
184184
onSubmit() {

packages/scripts/src/data-layer.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -220,17 +220,17 @@ export class DataLayer {
220220
dataLayerEvents.push("EN_FASTFORMFILL_ALL_FAILURE");
221221
}
222222

223-
// Push all collected events individually (GTM requirement)
224-
dataLayerEvents.forEach((event) => {
225-
this.dataLayer.push({ event });
226-
});
227-
228223
// Push all collected variables at once
229224
if (Object.keys(dataLayerData).length > 0) {
230225
dataLayerData.event = "pageJsonVariablesReady";
231226
this.dataLayer.push(dataLayerData);
232227
}
233228

229+
// Push all collected events individually (GTM requirement)
230+
dataLayerEvents.forEach((event) => {
231+
this.dataLayer.push({ event });
232+
});
233+
234234
this.attachEventListeners();
235235
}
236236

0 commit comments

Comments
 (0)