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

Opting in via set_status does not register the site #109

Open
TimothyBJacobs opened this issue Jan 25, 2024 · 0 comments · May be fixed by #111
Open

Opting in via set_status does not register the site #109

TimothyBJacobs opened this issue Jan 25, 2024 · 0 comments · May be fixed by #111
Labels
bug Something isn't working

Comments

@TimothyBJacobs
Copy link
Contributor

The docs advise creating a settings screen that toggles a site's optin status via Status::set_status().

https://github.com/stellarwp/telemetry#saving-opt-in-status-on-a-settings-page

In order to send data to the telemetry server a token value is required. This is only generated if the user opts in via the optin modal. If they skip in the initial modal, but then try to enable it via a setting, no data will be sent to the Telemetry server.

I'm not sure if this is a docs issue, and it is expected for plugins to call Opt_In_Subscriber::opt_in ourselves manually. To me, it seems like that should happen automatically.

I'm happy to create a PR, but need a bit of direction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
1 participant