Skip to content

feat: add fastapi integration#357

Closed
iloveitaly wants to merge 18 commits intoPostHog:masterfrom
iloveitaly:master
Closed

feat: add fastapi integration#357
iloveitaly wants to merge 18 commits intoPostHog:masterfrom
iloveitaly:master

Conversation

@iloveitaly
Copy link
Copy Markdown
Contributor

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Oct 28, 2025

Greptile encountered an error while reviewing this PR. Please reach out to support@greptile.com for assistance.

@rafaeelaudibert rafaeelaudibert requested a review from a team February 19, 2026 03:20
@rafaeelaudibert
Copy link
Copy Markdown
Member

We've updated our release process. We require sampo now. Please rebase on master and check README to understand what should be done.

@marandaneto
Copy link
Copy Markdown
Member

@dustinbyrne have you ever used fastapi? we prob need a sampe app here https://github.com/PostHog/posthog-python/tree/master/examples so we can test this out

@dustinbyrne
Copy link
Copy Markdown
Contributor

@iloveitaly this would be easier to review and merge if it had tests or an example as @marandaneto mentioned above. How would you like to proceed here? I'm happy to help out if you've moved on from this.

@marandaneto marandaneto marked this pull request as draft March 24, 2026 09:50
@marandaneto
Copy link
Copy Markdown
Member

@dustinbyrne we can prob test this once #279 gets merged

@marandaneto
Copy link
Copy Markdown
Member

@dustinbyrne we can prob test this once #279 gets merged

@iloveitaly since #470 got merged, can you make sure this works using that example? adding a test button/case etc

@marandaneto
Copy link
Copy Markdown
Member

closing because of this automation you have https://github.com/PostHog/posthog-python/pull/357/commits that pings us every day
happy to reopen and review again once we get this coment addressed

@marandaneto marandaneto closed this Apr 9, 2026
@iloveitaly
Copy link
Copy Markdown
Contributor Author

iloveitaly commented Apr 9, 2026

interesting, not sure what is causing that...

I don't have more time to invest into this, sorry.

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.

4 participants