elgentos / magento2-serversideanalytics

Server side analytics for Magento 2
MIT License
64 stars 21 forks source link

Changes for GA4 and removed UA #22

Closed JoostWan closed 1 year ago

JoostWan commented 1 year ago

More info about the session_id with GA4 can be found here

Can you check if it is working properly on your end as well?

barryvdh commented 1 year ago

Thanks! I've installed this patch. I see session id is stored in the last few orders, but will have to wait after the weekend to see the results.

barryvdh commented 1 year ago

Hmm I'm not sure if it's going correctly, I'm not seeing the data mapped to my users. I do see the conversions, just not connected to the users.

JoostWan commented 1 year ago

@barryvdh I think this is not possible at this moment because the limitations of the Measurement protocol for GA4.

Geographic information is only available via automatic collection from gtag, Google Tag Manager, or Google Analytics for Firebase.

Device information is only available via automatic collection from gtag, Google Tag Manager, or Google Analytics for Firebase.

https://developers.google.com/analytics/devguides/collection/protocol/ga4

WouterSteen commented 1 year ago

@JoostWan Are you willing to refactor this module a bit so we dont have to add another column to the quote/order table?

WouterSteen commented 1 year ago

Obv I would fully understand if you dont want to :) haha

Really appriciate the your input here!

I want to test this on a client who is asking for this, but cant right now because this db is like 20gb with order data, so when i put this to production the site is going to be offline for a while :)

Here is an article on what needs to be done: https://m.academy/blog/programmatically-add-column-magento-2-sales-order-database-table/

Ill look in to this later next week. Cheers!

WouterSteen commented 1 year ago

Working on it now! Hope ill finish today

WouterSteen commented 1 year ago

Almost done!

barryvdh commented 1 year ago

Do you have this working properly with alle the data matched? As discussed on Slack, we seemed to be missing source data etc from the transactions, so it looked like it didn't properly match. But not so familiar with GA4. We switched to client-side for now.

WouterSteen commented 1 year ago

Do you have this working properly with alle the data matched? As discussed on Slack, we seemed to be missing source data etc from the transactions, so it looked like it didn't properly match. But not so familiar with GA4. We switched to client-side for now.

In my new merge request this is working, as far as I could see.

But no production env yet, because we have a small bug in the pivot table. My team will work on this, as I'm on holiday next few weeks.

WouterSteen commented 1 year ago

@barryvdh @JoostWan ive pushed version 1.1.1. This contains fixes for GA4. This is been released in production today.

@JoostWan thanks for creating this MR!