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

fix(analytics): check do not track #4329

Merged
merged 1 commit into from
Aug 29, 2024
Merged

fix(analytics): check do not track #4329

merged 1 commit into from
Aug 29, 2024

Conversation

julienrbrt
Copy link
Member

Our do not track check was wrong and always returning true.
This is why we were getting nothing.

@julienrbrt julienrbrt added skip-changelog Don't check changelog for new entries backport/v28.x.y Backport to v28.x.y labels Aug 29, 2024
@github-actions github-actions bot added component:ci CI/CD workflow and automated jobs. component:configs type:internal labels Aug 29, 2024
@mergify mergify bot merged commit e04d0f6 into main Aug 29, 2024
47 of 49 checks passed
@mergify mergify bot deleted the julien/everyonednt branch August 29, 2024 08:33
mergify bot pushed a commit that referenced this pull request Aug 29, 2024
Our do not track check was wrong and always returning true.
This is why we were getting nothing.

(cherry picked from commit e04d0f6)
Pantani added a commit that referenced this pull request Aug 29, 2024
Our do not track check was wrong and always returning true.
This is why we were getting nothing.

(cherry picked from commit e04d0f6)

Co-authored-by: Julien Robert <[email protected]>
Co-authored-by: Danilo Pantani <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/v28.x.y Backport to v28.x.y component:ci CI/CD workflow and automated jobs. component:configs skip-changelog Don't check changelog for new entries type:internal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants