Skip to content

Comments

chore(flags): improved some logs for quota limiting#197

Merged
dmarticus merged 26 commits intomasterfrom
chore/more-quota-limit-logs
Mar 3, 2025
Merged

chore(flags): improved some logs for quota limiting#197
dmarticus merged 26 commits intomasterfrom
chore/more-quota-limit-logs

Conversation

@dmarticus
Copy link
Contributor

see this discussion here: PostHog/posthog-js#1758 (comment)

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR improves logging messages for feature flag quota limiting in the PostHog Python client, providing clearer guidance when users hit quota limits.

  • Added [FEATURE FLAGS] prefix to quota limit warning messages in client.py and request.py
  • Added explanatory text that feature flag data is being reset when quota limits are hit
  • Included a link to documentation about billing limits (https://posthog.com/docs/billing/limits-alerts)
  • Contains a merge conflict in CHANGELOG.md that needs resolution before merging (missing 3.15.1 release entry)

3 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

Copy link
Collaborator

@haacked haacked left a comment

Choose a reason for hiding this comment

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

:shipit:

@dmarticus dmarticus merged commit f4c9971 into master Mar 3, 2025
6 checks passed
@dmarticus dmarticus deleted the chore/more-quota-limit-logs branch March 3, 2025 19:00
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