Paths

Table of Contentst

Differential D6672

[native] Get `primaryIdentityPublicKey` before enabling `LogInPanel`
ClosedPublic
Actions

Authored by atul on Feb 9 2023, 11:27 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jun 14, 7:26 PM

	Unknown Object (File)
	Fri, Jun 14, 9:36 AM

	Unknown Object (File)
	Fri, Jun 14, 12:09 AM

	Unknown Object (File)
	Thu, Jun 13, 3:03 PM

	Unknown Object (File)
	Wed, Jun 12, 11:18 PM

	Unknown Object (File)
	Wed, Jun 12, 9:22 PM

	Unknown Object (File)
	Wed, Jun 12, 4:48 PM

	Unknown Object (File)
	Sun, Jun 9, 2:08 PM

Subscribers

None

Details

Reviewers

ashoat
tomek

Commits

rCOMM0130008ef475: [native] Get `primaryIdentityPublicKey` before enabling `LogInPanel`
rCOMMca051873c252: [native] Get `primaryIdentityPublicKey` before enabling `LogInPanel`

Summary

Make JSI call and get primaryIdentityPublicKey before enabled LogInPanel inputs/buttons.

Test Plan

Logged public key as expected.

Diff Detail

Repository

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

fix

Harbormaster completed remote builds in B16308: Diff 22308.Feb 9 2023, 11:43 AM

atul requested review of this revision.Feb 9 2023, 11:43 AM

Harbormaster completed remote builds in B16307: Diff 22307.Feb 9 2023, 11:43 AM

ashoat accepted this revision.Feb 9 2023, 11:49 AM

ashoat added inline comments.

native/account/log-in-panel.react.js
61 ↗	(On Diff #22308)	This isn't an "async function that hits server APIs" – maybe we should just remove those comments?

This revision is now accepted and ready to land.Feb 9 2023, 11:49 AM

atul added inline comments.Feb 9 2023, 11:55 AM

native/account/log-in-panel.react.js
61 ↗	(On Diff #22308)	Yeah without newlines splitting the `Props` into "chunks," I just assumed the comments referred to the prop immediately below. I'll remove the comments since I don't think they're particularly useful.

rebase + land

Harbormaster completed remote builds in B16310: Diff 22310.Feb 9 2023, 12:14 PM

Closed by commit rCOMMca051873c252: [native] Get `primaryIdentityPublicKey` before enabling `LogInPanel` (authored by atul). · Explain WhyFeb 9 2023, 12:21 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMca051873c252: [native] Get `primaryIdentityPublicKey` before enabling `LogInPanel`.

atul added a reverting change: rCOMM370d665778cc: Revert "[native] Get `primaryIdentityPublicKey` before enabling `LogInPanel`".Feb 9 2023, 12:37 PM

accidentally stashed changes responding to feedback instead of amending to commit..

This revision is now accepted and ready to land.Feb 9 2023, 12:38 PM

rebase

remove comments + land

This revision was landed with ongoing or failed builds.Feb 9 2023, 12:41 PM

Closed by commit rCOMM0130008ef475: [native] Get `primaryIdentityPublicKey` before enabling `LogInPanel` (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM0130008ef475: [native] Get `primaryIdentityPublicKey` before enabling `LogInPanel`.

Harbormaster completed remote builds in B16311: Diff 22312.Feb 9 2023, 12:55 PM

Harbormaster completed remote builds in B16312: Diff 22313.

Revision Contents
Changeset List

Path

Size

native/

account/

log-in-panel.react.js

21 lines

Diff 22307

native/account/log-in-panel.react.js

Loading...