Call the function in the Reports service instead of the keyserver.
The higher-order-function for callServerEndpoint will be removed in the next diff.
Depends on D9042
Details
Details
Ensured reporting is enabled in settings, then:
- On native, went to dev options and clicked "Trigger a crash" - got an error report in reports service.
- I sent some multimedia messages to generate media mission reports. They appeared in reports service.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| lib/actions/report-actions.js | ||
|---|---|---|
| 14 ↗ | (On Diff #30617) | Do you think we should have a similar timeout for the new call? |
| lib/actions/report-actions.js | ||
|---|---|---|
| 14 ↗ | (On Diff #30617) | Good idea. Do we have any helper utilities for timeouts? Fetch doesn't seem to have built-in support for them. We can use abortController for that, but since this fetch is in lib and can be called from anywhere, I need to check how it is supported across platforms, especially React Native |
| lib/actions/report-actions.js | ||
|---|---|---|
| 14 ↗ | (On Diff #30617) | I don't think we have some utils for this. In a couple of places, we're using Promise.race, but using AbortController sounds a lot better. |