Paths

Table of Contentst

Differential D6675

[native] Include `primaryIdentityPublicKey` in `register` request
ClosedPublic
Actions

Authored by atul on Feb 9 2023, 8:53 PM.

Tags

None

Referenced Files

	F33012143: D6675.1768371855.diff
	Tue, Jan 13, 6:24 AM

	F371985: 9b0832.png
	Feb 9 2023, 8:54 PM

Subscribers

None

Details

Reviewers

ashoat
tomek

Commits

rCOMMdd701e886ae8: [native] Include `primaryIdentityPublicKey` in `register` request

Summary

Include primaryIdentityPublicKey in register request from native.

Depends on D6674

Test Plan

Set breakpoint in accountCreationResponder:

Diff Detail

Repository

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Feb 9 2023, 8:53 PM

atul published this revision for review.Feb 9 2023, 8:54 PM

atul edited the test plan for this revision. (Show Details)

Harbormaster completed remote builds in B16314: Diff 22316.Feb 9 2023, 9:08 PM

resolve merge conflicts

rebase

Harbormaster completed remote builds in B16447: Diff 22484.Feb 13 2023, 6:31 PM

Harbormaster completed remote builds in B16450: Diff 22487.Feb 13 2023, 6:45 PM

ashoat added a parent revision: D6674: [keyserver] Modify `registerRequestInputValidator` to handle `primaryIdentityPublicKey`.Feb 13 2023, 7:00 PM

ashoat added a child revision: D6719: [web] Move `setPrimaryIdentityPublicKey` up from `SIWELoginForm` to `LoginForm`.

atul edited the summary of this revision. (Show Details)Feb 13 2023, 7:01 PM

atul mentioned this in D6719: [web] Move `setPrimaryIdentityPublicKey` up from `SIWELoginForm` to `LoginForm`.

ashoat accepted this revision.Feb 13 2023, 7:04 PM

ashoat added inline comments.

native/account/register-panel.react.js
333	Nit – I would destructure this and assign to a local to avoid repeating `this.props.`
484	Hmm... looks like this will add a spinner to the button, which is weird since it sort of implies that the request has been sent. It also blocks the `TextInput`, which isn't really necessary... I think we would just want to disable the button (Probably not a big deal since `commCoreModule.getUserPublicKey` should presumably be quite fast)

This revision is now accepted and ready to land.Feb 13 2023, 7:04 PM

destructure

atul added inline comments.Feb 14 2023, 12:11 AM

native/account/register-panel.react.js
484	Ah, I remember intending to add a `disabled` prop to `PanelButton`... I'll update this diff in place. Will also sneak in corresponding change to `LoginPanel`.

disable button instead of setting loadingStatus to loading if primaryIdentityPublicKey is null or undefined

This revision was landed with ongoing or failed builds.Feb 14 2023, 12:22 AM

Closed by commit rCOMMdd701e886ae8: [native] Include `primaryIdentityPublicKey` in `register` request. · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMdd701e886ae8: [native] Include `primaryIdentityPublicKey` in `register` request.

Harbormaster completed remote builds in B16463: Diff 22508.Feb 14 2023, 12:23 AM

Harbormaster completed remote builds in B16464: Diff 22509.Feb 14 2023, 12:34 AM

Revision Contents
Changeset List

Path

Size

lib/

types/

account-types.js

1 line

native/

account/

register-panel.react.js

28 lines

Diff 22487

lib/types/account-types.js

Loading...

native/account/register-panel.react.js

Loading...