Skip to content

fix(analytics-browser): auto-flush localhost uploads#1663

Draft
amp-opportunities wants to merge 2 commits intomainfrom
cursor/localhost-flush-and-errors-6b34
Draft

fix(analytics-browser): auto-flush localhost uploads#1663
amp-opportunities wants to merge 2 commits intomainfrom
cursor/localhost-flush-and-errors-6b34

Conversation

@amp-opportunities
Copy link
Copy Markdown

@amp-opportunities amp-opportunities commented Apr 9, 2026

Summary

  • enable immediate browser flush defaults on localhost and 127.0.0.1 unless the caller overrides them
  • surface upload failures through an optional onUploadError callback on browser config
  • add focused config/destination tests and a browser changelog note

Checklist

  • Does your PR title have the correct title format?
  • Does your PR have a breaking change?: No
Open in Web Open in Cursor 

cursoragent and others added 2 commits April 9, 2026 18:40
@cursoragent @amp-opportunities
fix(analytics-browser): auto-flush localhost uploads
500f01d 
Co-authored-by: amp-opportunities <amp-opportunities@users.noreply.github.com>
@cursoragent
test(analytics-browser): tidy config test imports
4ce30fc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@amp-opportunities @cursoragent