Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vue-gtag reporting #4856

Closed
yangwao opened this issue Jan 29, 2023 · 21 comments
Closed

vue-gtag reporting #4856

yangwao opened this issue Jan 29, 2023 · 21 comments
Labels
$ ~<50usd p1 preventing everyone from using app

Comments

@yangwao
Copy link
Member

yangwao commented Jan 29, 2023

Does ga works properly? As on CF web analytics are some numbers.

image

@yangwao yangwao added the p2 core functionality, or is affecting 60% of app label Jan 29, 2023
@roiLeo
Copy link
Contributor

roiLeo commented Jan 30, 2023

Something strange
Capture d’écran 2023-01-30 à 8 20 12 AM

@yangwao
Copy link
Member Author

yangwao commented Jan 30, 2023

I would always go on.

inspiration https://rarible.com/

image

New text could be

We use third-party 🍪 to make a better experience.

and cookies could be enabled by default, on "X" just disappear toast.

@roiLeo
Copy link
Contributor

roiLeo commented Jan 30, 2023

and cookies could be enabled by default, on "X" just disappear toast.

I don't agree with that. User won't be able to disable tracking services.

@yangwao
Copy link
Member Author

yangwao commented Jan 30, 2023

Then enable on "X" ? or keep accept?

@yangwao
Copy link
Member Author

yangwao commented Jan 31, 2023

Also drop could be no one clicks accept or ublock?

@yangwao
Copy link
Member Author

yangwao commented Feb 6, 2023

Notion on mobile has Accept All only. I would go this way.

IMG_20230206_114129_724.jpg

Ours could be made more mobile friendly?

IMG_20230206_114132_238.jpg

Same comment in

@yangwao
Copy link
Member Author

yangwao commented Feb 6, 2023

@roiLeo can we move on with this?

@roiLeo
Copy link
Contributor

roiLeo commented Feb 6, 2023

Notion on mobile has Accept All only. I would go this way.

We don't have cookie policy when it should be mandatory

Ours could be made more mobile friendly?

If you say so..

@roiLeo can we move on with this?

move on what? code won't change, user should have possibility to decline tracking

@yangwao
Copy link
Member Author

yangwao commented Feb 6, 2023

move on what? code won't change,

Dropped numbers means, that tracking does not turn on by default since now?
I just want to gain insight why it's happening showing 0.

CF dash showing 230 users on 2/2/2023 where GA 0.
Do you think all that 230 visits did not clicked accept or blocking tracking at all?

I also found some requirement for GA4 instead having UA (this we have i guess rn?)

image

@roiLeo
Copy link
Contributor

roiLeo commented Feb 7, 2023

Dropped numbers means, that tracking does not turn on by default since now?

=> default: false
=> accept: enable

I just want to gain insight why it's happening showing 0.
I also found some requirement for GA4 instead having UA (this we have i guess rn?)

could you share me GA4 config as I don't have access, I'll try to debug this.

CF dash showing 230 users on 2/2/2023 where GA 0.
Do you think all that 230 visits did not clicked accept or blocking tracking at all?

I don't know how CF track user, what data is stored? how long is it kept?
We need to inform user about these tracking services. (privacy policy)

@yangwao
Copy link
Member Author

yangwao commented Mar 20, 2023

=> accept: enable

that also means no-one clicks it, mmm

could you share me GA4 config as I don't have access, I'll try to debug this.

shared u admin to ....@gmail, there seems GA4 properties (?)

We need to inform user about these tracking services. (privacy policy)

either, probably their general ToS ?

I don't know how CF track user, what data is stored? how long is it kept?

@yangwao yangwao added $ ~<50usd p1 preventing everyone from using app and removed p2 core functionality, or is affecting 60% of app labels May 15, 2023
@yangwao
Copy link
Member Author

yangwao commented May 15, 2023

Can we have numbers back please?

@roiLeo
Copy link
Contributor

roiLeo commented May 16, 2023

Can we have numbers back please?

Seems to be an issue with nuxt-brigde and plugins

Btw #4928 is still needed..

edit:

@preschian
Copy link
Member

Seems to be an issue with nuxt-brigde and plugins

what if we implement the GA with a vanilla approach, without a plugin?

@roiLeo
Copy link
Contributor

roiLeo commented May 16, 2023

Seems to be an issue with nuxt-brigde and plugins

what if we implement the GA with a vanilla approach, without a plugin?

Yeah I thought about that one using useHead(), thing is I don't know if we can use publicRuntime to "hide" GA id

@preschian
Copy link
Member

does GA Id and UA- stuff are the same thing? seems like it will show it publicly

this is on coingecko
image

@roiLeo roiLeo mentioned this issue May 16, 2023
6 tasks
@roiLeo
Copy link
Contributor

roiLeo commented May 16, 2023

does GA Id and UA- stuff are the same thing? seems like it will show it publicly

Yep this is visible in source code, but shoud be set in a env var in order to set different code according to the domain (might be good in #6006)

@yangwao
Copy link
Member Author

yangwao commented May 16, 2023

set different code according to the domain

that would be nice actually have diff GA for each env, that would help filter out some numbers and cases

@yangwao
Copy link
Member Author

yangwao commented May 16, 2023

Btw #4928 is still needed..

full filled, ready to be added

@roiLeo
Copy link
Contributor

roiLeo commented May 16, 2023

full filled, ready to be added

@roiLeo
Copy link
Contributor

roiLeo commented May 16, 2023

Looking good
Screenshot 2023-05-16 at 14-06-07 Analytics Accueil

closed by #6006 , remind me to fix ga integration after Nuxt3 migration (#2690)

@roiLeo roiLeo closed this as completed May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
$ ~<50usd p1 preventing everyone from using app
Projects
None yet
Development

No branches or pull requests

3 participants