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

feat: Swap ingestion endpoints to new .i.posthog.com ones #20736

Merged
merged 4 commits into from
Apr 22, 2024

Conversation

benjackwhite
Copy link
Contributor

@benjackwhite benjackwhite commented Mar 6, 2024

Problem

We want to properly roll out the new endpoints. We will keep supporting old ones but this makes it clear for new users what they should use.

Changes

  • Updated all uses of the api host origin to show the appropriate host
  • Tried to find all app.posthog.com references that don't make sense anymore and update them to us.posthog.com
  • Updated the snippet to detect the new ingestion endpoint and swap to assets

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

How did you test this code?

Copy link
Contributor

github-actions bot commented Mar 6, 2024

Size Change: 0 B

Total Size: 1.01 MB

ℹ️ View Unchanged
Filename Size
frontend/dist/toolbar.js 1.01 MB

compressed-size-action

@posthog-bot
Copy link
Contributor

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week.

@benjackwhite benjackwhite added waiting Prevents stale-bot from marking the PR as stale. and removed stale labels Mar 15, 2024
@benjackwhite benjackwhite marked this pull request as ready for review March 15, 2024 11:59
Copy link
Member

@pauldambra pauldambra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@benjackwhite benjackwhite merged commit a8bcc67 into master Apr 22, 2024
107 checks passed
@benjackwhite benjackwhite deleted the feat/new-api-host branch April 22, 2024 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting Prevents stale-bot from marking the PR as stale.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants