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.

Sign up for GitHub

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

Jump to bottom

feat: use new refine telemetry api #346

Merged
merged 4 commits into from
Jun 8, 2023
Merged

feat: use new refine telemetry api #346

merged 4 commits into from
Jun 8, 2023

Conversation

BatuhanW
Copy link
Contributor

@BatuhanW BatuhanW commented Jun 8, 2023 •
edited
Loading

  • Removed node-analytics library
  • Removed config related to SEGMENT_KEY env variable
  • Use refine's api for tracking

@github-actions
Copy link

github-actions bot commented Jun 8, 2023

Hello from PR Helper

Is your PR ready for review and processing? Mark the PR ready by including #pr-ready in a comment.

If you still have work to do, even after marking this ready. Put the PR on hold by including #pr-onhold in a comment.

@omeraplak omeraplak merged commit b4fe1b1 into master Jun 8, 2023
7 checks passed
@omeraplak omeraplak deleted the new-telemetry branch June 8, 2023 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@omeraplak omeraplak omeraplak approved these changes

@aliemir aliemir Awaiting requested review from aliemir

Assignees
No one assigned
Labels
auto-merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@BatuhanW @omeraplak