Add `onFailure` Listener to Replace `observeEncryptedLogsUploadResult` Flow
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
- Issue type
- Feature
- Clarity
- Needs clarification
- Activity status
- Stale
- Tech stack
- kotlin
- Domain
- observability
Research direction
Start by locating the observeEncryptedLogsUploadResult flow and determine which proposed change is actually in scope: an onFailure listener, excluding no-internet events, or replacing Tracks with Sentry. The issue needs an agreed behavior and completion criteria before implementation can begin.
Written by the indexing model from the issue text.
Description
There are other ideas to make this whole observeEncryptedLogsUploadResult solution much better, like introducing an onFailure listener, exclude any "no internet connection" events, and replace Tracks with Sentry, which fits better to this kind of error reporting that team like JP/WPAndroid needs.
- Dominant language
- Kotlin
- Stars
- 3
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
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.
More from Automattic/EncryptedLogging
-
enhancement
Difficulty 4/5 3-5 days Newbie friendliness 35/100
-
bug
Difficulty 4/5 3-5 days Newbie friendliness 35/100
-
enhancement
Difficulty 4/5 3-5 days Newbie friendliness 25/100
All issues in Automattic/EncryptedLogging
Similar issues
-
Raijin Scans (FR)OpenBug Domain changed
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
keiyoushi/extensions-source#19644 · 1 reaction ·
Maintainers usually reply within 1 day
-
node: disconnection push notification is logged as "Unknown notification" instead of the disconnect warningPossibly taken @datatrigger claimed this 1 day ago. Openbug 🐞 Untriaged user issue
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
valkey-io/valkey-glide#7278 ·
Maintainers usually reply within 2 days
-
design
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement triaged
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
Maintainers usually reply within 3 days
-
Difficulty 1/5 1-3 hours Newbie friendliness 88/100
mobile-dev-inc/Maestro#3659 · 1 comment ·
Maintainers usually reply within 1 day