Details
Details
- Reviewers
tomek kamil will - Commits
- rCOMMb9bdf51320be: [lib] Introduce updateRelationshipSpec
For now, just Flow. I'll test this at a later point in the stack; for now, putting up diffs following the team's recent pattern to get feedback
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| lib/shared/dm-ops/update-relationship-spec.js | ||
|---|---|---|
| 103 | One thing I'm curious about is whether we should introduce a new queue for FIDs / AuxUserStore Arguably we've derisked the possibility of a scenario where the message comes in before FarcasterDataHandler has a chance to update the AuxUserStore, since we're relying on findUserIdentities calls instead of AuxUserStore | |
| lib/shared/dm-ops/update-relationship-spec.js | ||
|---|---|---|
| 103 | I think relying on findUserIdentities should be sufficient | |