Skip to content

Conversation

@gewenyu99
Copy link

@gewenyu99 gewenyu99 commented Nov 15, 2025

💡 Motivation and Context

We have generated references for Python and JS-based SDKs already. They've proven to be popular. This is a first pass at the iOS docs.

💚 How did you test it?

Generated specs, pointing strapi to this branch, viewed the docs it produced.

Preview the docs here: PostHog/posthog.com#13762

📝 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.

@gewenyu99
Copy link
Author

Screen.Recording.2025-11-14.at.9.28.55.PM.mov

Mostly working, broke the examples, tho :O

@gewenyu99
Copy link
Author

gewenyu99 commented Nov 15, 2025

Screen.Recording.2025-11-14.at.10.49.30.PM.mov

Got it working :OO

@ioannisj
Copy link
Collaborator

@gewenyu99 Can we open a draft PR on posthog.com pointing to this branch to see this in action?


if not params:
# No parameters - simple example
code = f"PostHogSDK.shared.{method_name}()"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This assumes only method calls on main instance PostHogSDK, but we also have some public methods on other types, for example config.setBeforeSend(_:)

@gewenyu99
Copy link
Author

@gewenyu99 Can we open a draft PR on posthog.com pointing to this branch to see this in action?

Yes, this need to actually go through Strapi, I'll set that up this week

This assumes only method calls on main instance PostHogSDK, but we also have some public methods on other types, for example config.setBeforeSend(_:)

Ooooo interesting, good call out

@gewenyu99 gewenyu99 marked this pull request as ready for review December 11, 2025 02:23
@gewenyu99 gewenyu99 requested a review from a team as a code owner December 11, 2025 02:23
@marandaneto
Copy link
Member

is there anything missing here besides making CI happy?

@gewenyu99
Copy link
Author

is there anything missing here besides making CI happy?

@marandaneto Need a review of the new documentation strings. I did a rough pass based on existing docs. Still need some tweaks.

Sorry for the stale PR! A lot happened last year. Will try to get this cleaned up soon.

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