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

Migrate docs posthog to proxy #502

Merged
merged 1 commit into from
Mar 12, 2024
Merged

Migrate docs posthog to proxy #502

merged 1 commit into from
Mar 12, 2024

Conversation

atbe
Copy link
Contributor

@atbe atbe commented Mar 12, 2024

Ellipsis 🚀 This PR description was created by Ellipsis for commit 005eb8e.

Summary:

This PR changes the api_host in the posthog.init function in main.html from 'https://app.posthog.com' to 'https://p.useinstructor.com', indicating a migration of the PostHog analytics service.

Key points:


Generated with ❤️ by ellipsis.dev

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me!

  • Reviewed the entire pull request up to 005eb8e
  • Looked at 14 lines of code in 1 files
  • Took 30 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 85%.
1. /docs/overrides/main.html:4:
  • Assessed confidence : 0%
  • Comment:
    Please ensure that the new URL 'https://p.useinstructor.com' is correctly configured to receive data from PostHog.
  • Reasoning:
    The change in the PR is a simple URL change for the PostHog analytics service. The new URL 'https://p.useinstructor.com' is being used instead of 'https://app.posthog.com'. This change doesn't seem to introduce any bugs or violate any best practices. However, it's important to ensure that the new URL is correct and the service is properly configured to receive data at this endpoint.

Workflow ID: wflow_w4QZUwLzraqVX1kz


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

@jxnl jxnl merged commit 4238940 into instructor-ai:main Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants