This differential move Android rescinds and badge updates to lib and makes the keyserver use shared code.
Details
Details
- Reviewers
tomek kamil - Commits
- rCOMM5c88cab342ee: Move android rescinds and badge updates to lib
- Flow
- Test that rescind and badge updates work on Android
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Comment Actions
I strongly encourage you to rethink the amount of type spreads you're doing, especially when they're nested within other types
| lib/push/notif-creators.js | ||
|---|---|---|
| 329 | In general, I am skeptical of this approach of nesting unions inside a spread. In the past I've seen Flow not handle it well. I usually opt to make the upper type a union, and to include the shared properties in each branch (senderDeviceID, platformDetails) via a spread | |
| 333 | ||
| lib/types/notif-types.js | ||
| 140 | What is this spread doing? | |
| 165 | What is this spread doing? | |
| lib/push/android-notif-creators.js | ||
|---|---|---|
| 266 ↗ | (On Diff #41556) | This is logic copy pasted from keyserver/push/rescind.js but extended to allow rescinds without notification id (thick thread rescinds). |
| 337 ↗ | (On Diff #41556) | This logic is copy-pasted from keyserver/src/send.js but extended to allow thick thread badge updates. |