Page MenuHomePhabricator
Differential D6085

[lib] Modify `siweAuth(...)` return type from `LogInResponse` to `LogInResult`
ClosedPublic
Actions

Authored by atul on Dec 29 2022, 1:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 10, 7:45 AM
Unknown Object (File)
Fri, Nov 8, 4:17 PM
Unknown Object (File)
Fri, Nov 8, 4:17 PM
Unknown Object (File)
Fri, Nov 8, 4:17 PM
Unknown Object (File)
Fri, Nov 8, 4:17 PM
Unknown Object (File)
Fri, Nov 8, 3:49 PM
Unknown Object (File)
Fri, Nov 8, 3:48 PM
Unknown Object (File)
Mon, Nov 4, 6:45 AM
View All Files
Subscribers
None

Details

Reviewers
ashoat
tomek
Commits
rCOMM55ce73655587: [lib] Modify `siweAuth(...)` return type from `LogInResponse` to `LogInResult`
Summary

We construct LogInResult using the request payload (SIWEAuthServerCall) and the respond (LogInResponse). Code was effectively cut/paste from lib/actions/user-actions:logIn.

Depends on D6077

Test Plan

Will be tested implicitly by subsequent diffs.

Diff Detail

Repository
rCOMM Comm
Branch
arcpatch-D6085 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul created this revision.Dec 29 2022, 1:23 AM
atul added a child revision: D6086: [lib] Add `siweAuthActionTypes.success` check everywhere there's a `logInActionTypes.success` check.
Harbormaster completed remote builds in B14880: Diff 20295.Dec 29 2022, 1:36 AM
atul requested review of this revision.Dec 29 2022, 1:36 AM
tomek accepted this revision.Dec 29 2022, 4:24 AM
tomek added inline comments.
lib/actions/siwe-actions.js
4–5 ↗(On Diff #20295)

We can merge these imports

This revision is now accepted and ready to land.Dec 29 2022, 4:24 AM
ashoat accepted this revision.Dec 29 2022, 9:24 AM
atul updated this revision to Diff 20396.Dec 29 2022, 2:00 PM

rebase before landing

atul updated this revision to Diff 20397.Dec 29 2022, 2:03 PM

address feedback from @tomek

atul added inline comments.Dec 29 2022, 2:04 PM
lib/actions/siwe-actions.js
4–5 ↗(On Diff #20295)

Thanks for catching that!

This revision was landed with ongoing or failed builds.Dec 29 2022, 2:07 PM
Closed by commit rCOMM55ce73655587: [lib] Modify `siweAuth(...)` return type from `LogInResponse` to `LogInResult` (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM55ce73655587: [lib] Modify `siweAuth(...)` return type from `LogInResponse` to `LogInResult`.
Harbormaster completed remote builds in B14951: Diff 20396.Dec 29 2022, 2:12 PM
Harbormaster completed remote builds in B14952: Diff 20397.Dec 29 2022, 2:16 PM

Revision Contents
Changeset List

PathSize
lib/
actions/
28 lines
1 line
types/
1 line
3 lines

Diff 20396

View Options

lib/actions/siwe-actions.js

Loading...
View Options

lib/actions/user-actions.js

Loading...
View Options

lib/types/account-types.js

Loading...
View Options

lib/types/redux-types.js

Loading...