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

[POC] of next13 support on top of @axiomhq/js #5

Closed
wants to merge 13 commits into from
Closed

[POC] of next13 support on top of @axiomhq/js #5

wants to merge 13 commits into from

Conversation

dasfmi
Copy link
Collaborator

@dasfmi dasfmi commented Jun 12, 2023

This is a POC of next13 logging and webivtals support on top of @axiomhq/js. This would help us better understand the requirements for the JS sdk and the challenges in supporting other layers on top.

Challenges / TODOs / Questions:

  • enforce error/warn if dataset name or token not resolved successfully.
  • catch client errors to stop breaking api functions or pages
  • test flushing
  • give the user the option to disable client batching
  • How to support client components without exposing axiom token?
  • do we handle the usage of navigator.becon in axiom/js?
  • how to handle the url for web-vitals? vercel integration has a special webVital endpoint

what would be the user-agent name? next-axiom/v or axiomjs-nextjs/v? or @axiomhq/nextjs/v

@vercel
Copy link

vercel bot commented Jun 12, 2023

@schehata is attempting to deploy a commit to the AxiomFM Team on Vercel.

To accomplish this, @schehata needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

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.

1 participant