Differential D11655

[lib] Introduce usePreRequestUserState
ClosedPublic
Actions

Authored by ashoat on Apr 12 2024, 9:14 PM.

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.Apr 12 2024, 9:14 PM

Herald added a subscriber: tomek. · View Herald TranscriptApr 12 2024, 9:14 PM

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

Harbormaster completed remote builds in B28203: Diff 39093.Apr 12 2024, 9:30 PM

ashoat requested review of this revision.Apr 12 2024, 9:30 PM

inka accepted this revision.Apr 15 2024, 7:44 AM

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

Closed by commit rCOMMfaa663cd9927: [lib] Introduce usePreRequestUserState. · Explain WhyApr 15 2024, 4:40 PM

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