segmentio / segmentio/analytics.js-integrations
[segmentio] Pull Request #4: Emit errors when the request fails
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 125
- Forks
- 136
- Avg merge
- 2h 45m
- Merged PRs (30d)
- 1
Description
Migrated from #4 by @gnandretta
I believe @cristiandouce has been talking with your support team about the details of the issues we are experiencing, but we need a mechanism to detect when a request to segment (via analytics.js) fails. The callbacks that can be provided to methods like track, page, identify and so on, don't yield any information about the status of the request.
One solution we came up with is to emit errors when the request fails and it is implemented in this pull request. Unfortunately, in order for this to actually work, yields/send-json needs to be updated. A pull request has already been sent. .
Contributor guide
No contributing guide indexed for this repository
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 reviewing the migrated pull request #4 to understand the proposed request-failure behavior, then inspect the referenced yields/send-json pull request #5. Done means requests made through analytics.js methods such as track, page, and identify expose failures through emitted errors.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- api
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100