[lib] Introduce usePreRequestUserState
ClosedPublic
Actions

Authored by ashoat on Fri, Apr 12, 2:14 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Apr 21, 8:12 PM

	Unknown Object (File)
	Sun, Apr 21, 2:47 PM

	Unknown Object (File)
	Sun, Apr 21, 2:43 PM

	Unknown Object (File)
	Sat, Apr 20, 3:07 PM

	Unknown Object (File)
	Thu, Apr 18, 10:54 PM

	Unknown Object (File)
	Thu, Apr 18, 12:28 PM

	Unknown Object (File)
	Tue, Apr 16, 2:33 PM

	Unknown Object (File)
	Sun, Apr 14, 1:08 PM

View All 10 Files

Subscribers

tomek

Details

Reviewers

inka

Commits

rCOMMfaa663cd9927: [lib] Introduce usePreRequestUserState

Summary

This should be a simple no-op refactor. I'm introducing this hook because in the next diff I'll be refactoring it.

Depends on D11654

Test Plan

Flow. Tested in combination with the next diff

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Fri, Apr 12, 2:14 PM

Herald added a subscriber: tomek. · View Herald TranscriptFri, Apr 12, 2:14 PM

ashoat added a child revision: D11656: [lib] Use useDerivedObject to reduce renders due to preRequestUserState.Fri, Apr 12, 2:15 PM

Harbormaster completed remote builds in B28203: Diff 39093.Fri, Apr 12, 2:30 PM

ashoat requested review of this revision.Fri, Apr 12, 2:30 PM

inka accepted this revision.Mon, Apr 15, 12:44 AM

This revision is now accepted and ready to land.Mon, Apr 15, 12:44 AM

Closed by commit rCOMMfaa663cd9927: [lib] Introduce usePreRequestUserState (authored by ashoat). · Explain WhyMon, Apr 15, 9:40 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMfaa663cd9927: [lib] Introduce usePreRequestUserState.

Revision Contents
Changeset List

Path

Size

lib/

actions/

user-actions.js

8 lines

selectors/

account-selectors.js

7 lines

Diff 39131

View Options

lib/actions/user-actions.js