[web] Fetch loggedInUserInfo in useExistingThreadInfoFinder
ClosedPublic
Actions

Authored by ashoat on Feb 17 2023, 6:56 AM.

Details

Reviewers

Commits

rCOMM77968cefadfd: [web] Fetch loggedInUserInfo in useExistingThreadInfoFinder

Summary

This diff should be a no-op. In a later diff, I'm going to update createPendingThread so that it needs the viewer's username, as we may need to fetch their ENS name.

Depends on D6755

Test Plan

Flow, and playing around with pending threads on web

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Feb 17 2023, 6:56 AM

Herald added a subscriber: tomek. · View Herald TranscriptFeb 17 2023, 6:56 AM

ashoat added a child revision: D6757: [native] Convert ComposeThreadButton to a hook.Feb 17 2023, 6:57 AM

Harbormaster completed remote builds in B16573: Diff 22674.Feb 17 2023, 7:11 AM

ashoat requested review of this revision.Feb 17 2023, 7:11 AM

atul accepted this revision.Feb 20 2023, 1:15 PM

This revision is now accepted and ready to land.Feb 20 2023, 1:15 PM

Closed by commit rCOMM77968cefadfd: [web] Fetch loggedInUserInfo in useExistingThreadInfoFinder (authored by ashoat). · Explain WhyFeb 20 2023, 3:13 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM77968cefadfd: [web] Fetch loggedInUserInfo in useExistingThreadInfoFinder.

Revision Contents
Changeset List

Path

Size

lib/

shared/

thread-utils.js

10 lines

Diff 22765

View Options

lib/shared/thread-utils.js

[web] Fetch loggedInUserInfo in useExistingThreadInfoFinderClosedPublicActions