Paths

Table of Contentst

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

Authored by atul on Dec 29 2022, 9:23 AM.

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Dec 29 2022, 9: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, 9:36 AM

atul requested review of this revision.Dec 29 2022, 9:36 AM

tomek accepted this revision.Dec 29 2022, 12:24 PM

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, 12:24 PM

ashoat accepted this revision.Dec 29 2022, 5:24 PM

rebase before landing

address feedback from @tomek

atul added inline comments.Dec 29 2022, 10: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, 10:07 PM

Closed by commit rCOMM55ce73655587: [lib] Modify `siweAuth(...)` return type from `LogInResponse` to `LogInResult`. · 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, 10:12 PM

Harbormaster completed remote builds in B14952: Diff 20397.Dec 29 2022, 10:16 PM

Revision Contents
Changeset List

Path

Size

lib/

actions/

siwe-actions.js

30 lines

user-actions.js

1 line

types/

account-types.js

1 line

redux-types.js

3 lines

Diff 20398

View Options

lib/actions/siwe-actions.js

View Options

lib/actions/user-actions.js

View Options

lib/types/account-types.js

View Options

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

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 20398

lib/actions/siwe-actions.js

lib/actions/user-actions.js

lib/types/account-types.js

lib/types/redux-types.js

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

Revision Contents
Changeset List