Skip to content

Exclude posthog traffic#5342

Merged
andrzejewsky merged 2 commits intomainfrom
merx-1362-posthog-prod-only
Jan 10, 2025
Merged

Exclude posthog traffic#5342
andrzejewsky merged 2 commits intomainfrom
merx-1362-posthog-prod-only

Conversation

@andrzejewsky
Copy link
Contributor

Scope of the change

  • Currently we push events to the posthog from various of our environments even if we don't need that (traffic is excluded in posthog).
  • This PR disables that and allows to specify excluded domains on releases

@changeset-bot
Copy link

changeset-bot bot commented Jan 9, 2025

🦋 Changeset detected

Latest commit: ce937f3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
saleor-dashboard Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot temporarily deployed to pr-5342 January 9, 2025 10:36 Destroyed
@andrzejewsky andrzejewsky marked this pull request as ready for review January 9, 2025 11:14
@andrzejewsky andrzejewsky requested a review from a team January 9, 2025 11:14
@andrzejewsky andrzejewsky requested a review from a team as a code owner January 9, 2025 11:14
@andrzejewsky andrzejewsky added the run pw-e2e Run e2e (basic suite from PR automation) label Jan 9, 2025
@NyanKiyoshi NyanKiyoshi changed the title Exclude pothog traffic Exclude posthog traffic Jan 9, 2025
@github-actions github-actions bot temporarily deployed to pr-5342 January 9, 2025 11:17 Destroyed
@github-actions github-actions bot temporarily deployed to pr-5342 January 10, 2025 09:31 Destroyed
@github-actions github-actions bot temporarily deployed to pr-5342 January 10, 2025 09:47 Destroyed
@andrzejewsky andrzejewsky merged commit e6e3f6b into main Jan 10, 2025
13 of 14 checks passed
@andrzejewsky andrzejewsky deleted the merx-1362-posthog-prod-only branch January 10, 2025 12:01
poulch added a commit that referenced this pull request Jan 10, 2025
Co-authored-by: Paweł Chyła <chyla1988@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run pw-e2e Run e2e (basic suite from PR automation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants