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

Phoenix LiveView instrumentation #500

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

binaryseed
Copy link
Collaborator

This PR adds Phoenix LiveView specific instrumentation.

  • Collect Transactions for the initial LiveView WebSocket lifecycle (ie: the first "live" mount ... render phase)
    • (The LiveView static html page load is already collected by Phoenix/Plug instrumentation)
  • Record spans for each component of the LiveView lifecycle in all LiveView Transactions

Closes #485

tpitale
tpitale previously approved these changes Feb 13, 2025
@tpitale
Copy link
Contributor

tpitale commented Feb 14, 2025

@binaryseed If you're ready to merge, let's go for it so we can get a release out. I'm on vacation for a week starting today 😬

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.

Extended Live View support
2 participants