Skip to content

Conversation

@iamnutzy1975
Copy link

I had installed GA4Dataform for 7 different GA4 properties and I was commonly seeing errors with assertions due to session valadility and timeliness. I tracked it down to "fresh" tables and APP sessions.

I tested these changes by overriding the CORE assertion file and creating my own CUSTOM version where I installed GA4 Data Form - fixed and no issues were the result.

Therefore, I decide to share my fixes so that hopefully that make it into the CORE product.

Added condition for APP sourced sessions.  APP sessions don't need to have landing page (as WEB based sessions do), but they should always have an APP Id
Added condition to elimnate fresh table stream suffixes being included in date compare (timeliness assertion)
@zjuul
Copy link

zjuul commented May 1, 2025

@iamnutzy1975 thank you for this patch!
The current assertions are indeed assuming no fresh tables exist, and that there is always something as landing_page.

Also: thank you for your patience! This is the first outside code that is contributed to us, and it took us a while to find time to build a process around this.

Are you OK with signing a Contributor License Agreement?
Explanation can be found here: https://ga4dataform.com/contributing-to-ga4dataform/

This is a super small patch, but size is not always important - we value your contribution greatly.
You won't have to sign it for every contribution, just once is enough. After that, you're an Official Contributor to the project :)

@iamnutzy1975
Copy link
Author

iamnutzy1975 commented May 2, 2025 via email

@zjuul
Copy link

zjuul commented May 26, 2025

Hi Gord, I totally missed your reply - somehow I did not receive a notification and the attachment is not attached, it is sent straight to github and discarded there(?)

Can you please email it directly to [email protected] ?

Thank you so much for your patience!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants