microsoft / microsoft/ApplicationInsights-JS
Move AJAX hookup logic to snippet
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 685
- Forks
- 261
- Avg merge
- 21h 33m
- Merged PRs (30d)
- 5
Description
Currently AI JS SDK is subscribing for AJAX requests after full JS from CDN has loaded, which means that we are likely missing initial AJAX requests that are happening before first page load.
Suggestion is to refactor it to move the subscribing code to snippet, which would help us miss fewer requests. Opening this issue to see if generates support feedback/support from community so we can decide how to prioritize it.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the snippet and the code path that loads the full JavaScript SDK from the CDN, then trace where AJAX request subscription currently begins. The change is complete when subscription starts from the snippet early enough to capture more initial AJAX requests, with the relevant behavior verified by the project's existing tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, typescript
- Domain
- observability
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100