Skip to content
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

Adds posthog #34

Merged
merged 8 commits into from
Jun 24, 2024
Merged

Adds posthog #34

merged 8 commits into from
Jun 24, 2024

Conversation

Adebesin-Cell
Copy link
Member

Adds posthog

Adds posthog to braindao-ui

Fixes EveripediaNetwork/issues#2810

Copy link

vercel bot commented Jun 21, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
braindao-ui ✅ Ready (Inspect) Visit Preview Jun 24, 2024 11:04am

Copy link

stackblitz bot commented Jun 22, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@@ -13,7 +14,9 @@ const client = createConfig({
const ClientProviders = ({ children }: PropsWithChildren) => {
return (
<ThemeProvider attribute="class" defaultTheme="dark">
<WagmiConfig config={client}>{children}</WagmiConfig>
<WagmiConfig config={client}>
Copy link
Member

Choose a reason for hiding this comment

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

not related but why do we have have a wagmi wrapper in this static page?

Copy link
Member Author

Choose a reason for hiding this comment

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

I saw some contract read, not sure why but I’ll check what they do :))

Copy link
Member Author

Choose a reason for hiding this comment

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

Screenshot 2024-06-23 at 5 07 55 PM

It's used for the stat pointers. I think we can remove it and use fetch calls since it's use a little potion of the app.

Copy link
Member

Choose a reason for hiding this comment

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

Also, lets rename this file to ClientProviders.ts and move it inside src/components/layouts

@kesar
Copy link
Member

kesar commented Jun 24, 2024

lets merge it. ensure we add the proper envs as well 👍🏻 @Adebesin-Cell

@Adebesin-Cell
Copy link
Member Author

lets merge it. ensure we add the proper envs as well 👍🏻 @Adebesin-Cell

Yes, added the keys, was waiting for an approval earlier, would merge now, thanks!

@Adebesin-Cell Adebesin-Cell merged commit 2ee87d4 into main Jun 24, 2024
2 checks passed
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.

Integrate Posthog analytics into BrainDAO
3 participants