-
Notifications
You must be signed in to change notification settings - Fork 59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add "User Interview" prompts #1894
Conversation
I was hoping to have the popup appear in the top-right, but that would require us to build a custom popup and isn't worth the hassle at the moment. |
Had a call with Luke, Product Manager for the Interview work in PostHog and he took onboard some feedback, that he said they'd be able to implement in the next few weeks. Won't require any code changes, all PostHog-side, but will give.us a little more control on positioning and styling. |
ooh - very good point - I don't have that enabled localy, let me configure |
PostHog doesn't allow for moving that pop-up at the moment, unless we build our own custom popup (is plausible but half a day, to a days work). Not sure we have much control over the HubSpot one either do we, i.e. we can't delay it appearing if the PostHog widget is present, etc) |
ed04de8
to
4bb8447
Compare
went ahead and built our own popup instead, was fairly straight forward to implement and can be configured via the feature flag
Config options inside PostHog:Example Popup In-AppPostHog DocumentationIt's built using the recommended approach defined in PostHog's in-app documentation: Including the setting of a |
@Pezmc fyi ^ - happy to talk you through this if you'd like, main things to know:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joepavitt Ran me though setting this up locally, we did some extra click testing and confirmed all is working (minus two small edge cases fixed above)!
Description
Enable PostHog Apps such that the Interviewer app can generate the popup. All configuration done within PotHog to control:
Update
See #1894 (comment)
Related Issue(s)
Closes #1828
Checklist
flowforge.yml
?flowforge/helm
to update ConfigMap Templateflowforge/CloudProject
to update values for Staging/ProductionLabels
backport
labelarea:migration
label