Skip to content

feat: add fastapi integration#357

Closed
iloveitaly wants to merge 18 commits into
PostHog:masterfrom
iloveitaly:master
Closed

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

Conversation

@iloveitaly

Copy link
Copy Markdown
Contributor

@greptile-apps

greptile-apps Bot commented Oct 28, 2025

Copy link
Copy Markdown
Contributor

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

iloveitaly commented Apr 9, 2026

Copy link
Copy Markdown
Contributor Author

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