Skip to content

fix: default api host when omitted#108

Merged
marandaneto merged 1 commit intoPostHog:mainfrom
marandaneto:fix/default-api-host
Apr 30, 2026
Merged

fix: default api host when omitted#108
marandaneto merged 1 commit intoPostHog:mainfrom
marandaneto:fix/default-api-host

Conversation

@marandaneto
Copy link
Copy Markdown
Member

💡 Motivation and Context

api_host had a default value internally, but the config schema still required users to provide it. This makes api_host optional so SDK initialization can use the default PostHog host when omitted.

💚 How did you test it?

  • mix test test/posthog/config_test.exs

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • I updated the docs if needed.
  • No breaking change or entry added to the changelog.

If releasing new changes

  • Ran sampo add to generate a changeset file
  • Added the release label to the PR

@marandaneto marandaneto marked this pull request as ready for review April 30, 2026 12:13
@marandaneto marandaneto requested review from a team and rafaeelaudibert as code owners April 30, 2026 12:13
@marandaneto marandaneto merged commit 48baad3 into PostHog:main Apr 30, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants