Paths

Table of Contentst

Differential D11644

Extract shareable logic from CreateSIWEBackupMessage component
ClosedPublic
Actions

Authored by marcin on Fri, Apr 12, 5:23 AM.

Tags

None

Referenced Files

	F1694009: D11644.id39404.diff
	Thu, May 2, 8:27 PM

	F1693979: D11644.id39413.diff
	Thu, May 2, 8:12 PM

	Unknown Object (File)
	Sat, Apr 27, 5:39 PM

	Unknown Object (File)
	Fri, Apr 26, 4:19 PM

	Unknown Object (File)
	Fri, Apr 26, 1:31 PM

	Unknown Object (File)
	Mon, Apr 22, 11:42 AM

	Unknown Object (File)
	Mon, Apr 22, 9:38 AM

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

View All 17 Files

Subscribers

Details

Reviewers

tomek
atul
inka

Commits

rCOMMc1e55f55b290: Extract shareable logic from CreateSIWEBackupMessage component

Summary

This differential refactors CreateSIWEBackupMessage component so that most of its code and logic can be resused. It is necessary since next diff will
introduce new component that renders the same UI to already logged in SIWE users to create backup message if it is not in the database.

Test Plan

Flow
Test that already introduced functionality wasn't broken.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

marcin created this revision.Fri, Apr 12, 5:23 AM

Herald added a subscriber: ashoat. · View Herald TranscriptFri, Apr 12, 5:23 AM

marcin added a parent revision: D11638: Persist SIWE backup secrets to SQLite after succesfull registration process.Fri, Apr 12, 5:23 AM

Harbormaster completed remote builds in B28186: Diff 39075.Fri, Apr 12, 5:39 AM

marcin requested review of this revision.Fri, Apr 12, 5:39 AM

marcin added a child revision: D11645: Force already logged in ETH users to create and sign backup message.Fri, Apr 12, 5:41 AM

inka accepted this revision.Fri, Apr 12, 6:54 AM

This revision is now accepted and ready to land.Fri, Apr 12, 6:54 AM

ashoat added inline comments.Fri, Apr 12, 8:53 AM

native/account/registration/siwe-backup-message-creation.react.js
165 ↗	(On Diff #39075)	Nit: I'd reverse the order of these, to match the order in which they are defined in the file

Memoize CreateSIWEBackupMessageBase

Harbormaster completed remote builds in B28226: Diff 39126.Mon, Apr 15, 8:38 AM

marcin mentioned this in D11636: Implement screen to create backup mesage and its signature.Tue, Apr 16, 5:04 AM

Use backup icon from MaterialIcons

Harbormaster completed remote builds in B28239: Diff 39145.Tue, Apr 16, 5:25 AM

REbase to reflect changes from parent diff

Harbormaster completed remote builds in B28286: Diff 39210.Thu, Apr 18, 1:40 AM

Rebase before landing

Harbormaster completed remote builds in B28408: Diff 39404.Tue, Apr 23, 6:15 AM

Closed by commit rCOMMc1e55f55b290: Extract shareable logic from CreateSIWEBackupMessage component (authored by marcin). · Explain WhyTue, Apr 23, 6:54 AM

This revision was automatically updated to reflect the committed changes.

marcin added a commit: rCOMMc1e55f55b290: Extract shareable logic from CreateSIWEBackupMessage component.

Revision Contents
Changeset List

Path

Size

native/

account/

registration/

registration-navigator.react.js

2 lines

siwe-backup-message-creation.react.js

190 lines

Diff 39413

native/account/registration/registration-navigator.react.js

Loading...

native/account/registration/siwe-backup-message-creation.react.js

Loading...