Currently the datalayer pushes are not using events
This is generally considered bad practice, also prevents the data easily being reused for other purposes and most importantly improves the ability to debug the implementation.
Its important that the above events aren't similar to the magento events used in their Enterprise commerce tag manager implementation- hence I suggested above to make sure they are different enough.
jissereitsma
commented
5 years ago
Currently the datalayer pushes are not using events
This is generally considered bad practice, also prevents the data easily being reused for other purposes and most importantly improves the ability to debug the implementation.
eg productdetail should be
dataLayer.push({'event':'prodDetail', ecommerce: <ecom object>})Other events could be
Its important that the above events aren't similar to the magento events used in their Enterprise commerce tag manager implementation- hence I suggested above to make sure they are different enough.