PostHog Slack App Product Page#17077
Conversation
|
@joshsny @VojtechBartos tagging you in to review content! |
Deploy preview
|
The merge of master into posthog-slack-app put the Korean shell components back under src/pages/ko/components/, which causes gatsby-plugin-page-creator to treat lottieAnimations.ts as a page component and abort the build. Master fixed this in #17081 by moving the components to src/components/Korean/; re-apply that fix here. Generated-By: PostHog Code Task-Id: c19842ec-5e44-4fe4-9804-ae103292b6be
VojtechBartos
left a comment
VojtechBartos
left a comment
There was a problem hiding this comment.
Looks great, left just couple of comments
| commands from your colleagues). | ||
| </li> | ||
| <li> | ||
| It watches CI, reruns failed jobs that look environmental, and doesn't touch workflow files. |
There was a problem hiding this comment.
q @joshsny: do we wanna go to beta with babysitting feature enabled?
corywatilo
left a comment
corywatilo
left a comment
There was a problem hiding this comment.
Visual feedback
- Since this is a marketing page, not sure Connect Slack is the right CTA since we don't know if the user already has an account. But fine leaving it in if we denote it with Already using PostHog? [Connect Slack]
- Code / PostHog AI
- The product titles link to product pages but they need
state={{newWindow: true}}to open in a new window instead of taking over the current window - Ask Claude to make these stack for 1-2 container query sizes so these don't switch to columns until there's enough space
- The product titles link to product pages but they need
- This block has some left-margin which should only apply once it switches to floating, as it's only needed once floated
- Might need a solid background color. (I'd try
bg-light dark:bg-darkorbg-accentshould handle both cases maybe
- Remove the window sizing/positioning added to
@src/context/App.tsxentirely. This causes the window to resize if you're clicking between different other pages in the left nav. - Have it underline PostHog Code and should use
text-primary - These icons have issues, not entirely sure they're necessary (as they don't directly associate with the text), or they could go in two columns with simple green checkboxes instead
- Building your own? PostHog MCP... link also needs
state={{newWindow: true}}. Also See the PostHog Code Slack docs in the FAQs and a few other links in FAQs - Dark mode image could use a background color in dark mode (like
bg-lightmost likely)
Content suggestions
Is this supposed to say, "Don't at me, ask @PostHog? Currently reads like you're telling PostHog not to @ you
then prompt a PR to do something about it.
then ask PostHog to create a PR to fix it – without ever leaving Slack.
Deliver product data to people who care
the "who care" part reads a bit marketing-y. Maybe something about how it gives you helpful insights in your existing workflow so you don't have to go find it somewhere else, etc? Like it pushes it to where you already look for it.
page tweaks, new tiles, it follows
page tweaks, new tiles – it follows
We should also definitely have a look into why all the Korean changes are in here before merging.
3 participants
Changes
@corywatilo shooting my shot here as discussed last week
Started in a google doc but as I worked on the slack UI mockups for anchor images I gravitated to building the page itself in PostHog Code.
Preview build
Checklist
vercel.json