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

Posthog analytics Integration for iq.wiki backend #581

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

icemedia001
Copy link
Contributor

Integrate Posthog analytics for iq.wiki backend

Replaced google analytics with posthog
Added posthog API to iq.wiki backend

On graphql playground, after connecting to server input:

mutation {
viewEvent(event: "testinganew\event", distinctId: "someuser101", properties: { key1: "value1" })
}

closes EveripediaNetwork/issues#2817

@icemedia001 icemedia001 requested review from kesar and AshakaE September 1, 2024 13:41
Copy link

stackblitz bot commented Sep 1, 2024

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

@icemedia001 icemedia001 mentioned this pull request Sep 1, 2024
@kesar
Copy link
Member

kesar commented Sep 1, 2024

how did u test this implementation? can u share some screenshots?

@kesar
Copy link
Member

kesar commented Sep 1, 2024

also checks are failing, before asking for a review you should ensure you dont have errors in the PR

@icemedia001
Copy link
Contributor Author

also checks are failing, before asking for a review you should ensure you dont have errors in the PR

Noted sir.
I'll work on it now.

@kesar
Copy link
Member

kesar commented Sep 1, 2024

also checks are failing, before asking for a review you should ensure you dont have errors in the PR

Noted sir. I'll work on it now.

any update?

@@ -7,9 +9,20 @@ import ActivityModule from '../App/Activities/activity.module'
import AppService from '../App/app.service'

@Module({
imports: [httpModule(10000), ActivityModule],
imports: [httpModule(10000), ActivityModule,
PosthogModule.forRootAsync({
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 4 locations. Consider refactoring.

.getOne())

const [founders, blockchain] = await Promise.all([
(async () => {
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 2 locations. Consider refactoring.

}
return []
})(),
(async () => {
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 2 locations. Consider refactoring.

Copy link

codeclimate bot commented Sep 23, 2024

Code Climate has analyzed commit fc9b3d7 and detected 11 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 4
Duplication 7

View more on Code Climate.

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 for iq.wiki backend
2 participants