Page MenuHomePhabricator
Differential D11911

[lib] hooks for reserved registration
AbandonedPublic
Actions

Authored by varun on May 6 2024, 7:29 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 22, 11:02 PM
Unknown Object (File)
Thu, Nov 14, 12:37 PM
Unknown Object (File)
Thu, Nov 14, 9:49 AM
Unknown Object (File)
Tue, Nov 12, 12:10 PM
Unknown Object (File)
Tue, Nov 12, 9:48 AM
Unknown Object (File)
Tue, Nov 12, 5:18 AM
Unknown Object (File)
Sun, Nov 3, 10:33 PM
Unknown Object (File)
Oct 20 2024, 5:05 PM
View All 78 Files
Subscribers
None

Details

Reviewers
ashoat
tomek
Summary

these aren't really necessary, but in the next diff, i was getting a flow error when i tried using the identity client methods directly

Depends on D11910

Test Plan

tested from js later in stack by calling the useLogIn hooks with a reserved username and reserved wallet address and confirming that i was registered on identity

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun created this revision.May 6 2024, 7:29 AM
varun added a child revision: D11912: [lib] add reserved username workflow to identity login actions.May 6 2024, 7:33 AM
varun added inline comments.May 6 2024, 7:37 AM
lib/actions/user-actions.js
552

i'll separate these hooks from identityRegisterActionTypes a little, because they're not actually intended to be dispatched with those action types

1012–1014

we actually don't need to expose these new hooks publicly, since they'll only be called by the identity login hooks

varun published this revision for review.May 6 2024, 7:48 AM
varun planned changes to this revision.
Harbormaster completed remote builds in B28708: Diff 39857.May 6 2024, 7:57 AM
varun abandoned this revision.May 31 2024, 12:55 PM

we dont need these

Revision Contents
Changeset List

PathSize
lib/
actions/
31 lines

Diff 39857

View Options

lib/actions/user-actions.js

Loading...