posthog
posthog copied to clipboard
PostHog
feat: Ability to show plugin developers commitHash in UI
Problem
Plugin developers to test in our cloud need to have us update the plugin, there's no way for them to see what version it's running, so sometimes they will ask, instead we can use log level 0 for them and include that info in the setup message.
Request: https://posthog.slack.com/archives/C05M3SMFSAG/p1715085765245699
Changes
- added commitHash to pod logs and to plugin setup log
- can update plugin in django admin (blank=True), since it's manyToMany it doesn't change anything
- can override logLEvel to 0 (|| -> ??)
- Removed single organization self-replication allowed since the app has been disabled for them (unrelated clean-up, just because I was there)
👉 Stay up-to-date with PostHog coding conventions for a smoother review.
Does this work well for both Cloud and self-hosted?
How did you test this code?
saw in logs:
🔌 setupPlugin succeeded for plugin Taxonomy Plugin ID 4 (organization ID 018e628e-99d7-0000-ff0b-1c5490bdc38f - global - tag 21215eace23e3a89058cf1774dc54c65b4c96dff).
A test needs updating: https://github.com/PostHog/posthog/actions/runs/9018992960/job/24780853603?pr=22225#step:14:281
This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week.
This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week.
This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week.
This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week.
This PR was closed due to lack of activity. Feel free to reopen if it's still relevant.