Details
Search for users and check on the icon:
Made sure that users which have Local DM are registered to Identity, other users are either reserved usernames or only from my local keyserver
Diff Detail
- Repository
- rCOMM Comm
- Branch
- publish-personal-2
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| lib/actions/user-actions.js | ||
|---|---|---|
| 1381–1392 | I had to move it because of the dependency cycle | |
| lib/hooks/thread-search-hooks.js | ||
| 54–59 | this is called when usingCommServicesAccessToken = false so we're not able to use thick threads anyway | |
| lib/shared/thread-utils.js | ||
| 1713–1725 | Not the cleanest solution but reduce amount of required changes | |
| lib/hooks/user-identities-hooks.js | ||
|---|---|---|
| 24 | ||
I feel like this would have been much easier to review if it had been split up more
| lib/actions/user-actions.js | ||
|---|---|---|
| 1381–1392 | This should've been a separate diff | |
| lib/hooks/thread-search-hooks.js | ||
| 90 | Invert condition | |
| lib/hooks/user-identities-hooks.js | ||
| 11–22 | Combining this here feels risky for dependency cycles. I think that separating this out into its own file is the best way to avoid getting the dependency cycle again | |
| 26 | I'd consider returning a set here... it feels like that would be more convenient for callers than an array | |
| 37 | ||
| lib/shared/thread-utils.js | ||
| 1714 | ||