Paths

Table of Contentst

Differential D7657

Establish olm notifs session on both sides during registration
ClosedPublic
Actions

Authored by marcin on Apr 27 2023, 1:40 PM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

tomek
atul
bartek
ashoat

Commits

rCOMM7cbc1822445f: Establish olm notifs session on both sides during registration
rCOMM29f99ee4dfad: Establish olm notifs session on both sides during registration

Summary

This differential implements olm notifs session establishment during account creation.

Test Plan

Create new account. Ensure new entry in olm_sessions table appears

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

marcin created this revision.Apr 27 2023, 1:40 PM

Herald added a subscriber: ashoat. · View Herald TranscriptApr 27 2023, 1:40 PM

Harbormaster completed remote builds in B18859: Diff 25828.Apr 27 2023, 1:58 PM

marcin requested review of this revision.Apr 27 2023, 1:58 PM

marcin added a parent revision: D7654: Establish olm sessions for notifs on both sides during log-in.Apr 27 2023, 2:02 PM

marcin added a child revision: D7669: Establish olm notifs sessions during siwe account creation.Apr 27 2023, 3:15 PM

ashoat requested changes to this revision.Apr 27 2023, 8:10 PM

ashoat added inline comments.

keyserver/src/creators/account-creator.js
132–138 ↗	(On Diff #25828)	See feedback in D7654. Please be more thoughtful when introducing new `await` keywords
native/account/register-panel.react.js
295–297 ↗	(On Diff #25828)	See feedback in D7654. Please be more thoughtful when introducing new `await` keywords

This revision now requires changes to proceed.Apr 27 2023, 8:10 PM

Better usage of awaits. Ensure cookie iscreated prior to olm session.

Harbormaster completed remote builds in B18989: Diff 26006.May 2 2023, 3:57 PM

ashoat accepted this revision.May 2 2023, 5:13 PM

This revision is now accepted and ready to land.May 2 2023, 5:13 PM

Rebase

Harbormaster completed remote builds in B19286: Diff 26399.May 11 2023, 1:11 PM

Rebase before landing

Harbormaster completed remote builds in B19561: Diff 26742.May 22 2023, 9:15 AM

Rename call to get and rebase before landing

Harbormaster completed remote builds in B19570: Diff 26751.May 22 2023, 10:47 AM

Rebase before landing

Harbormaster completed remote builds in B19842: Diff 27173.May 29 2023, 7:02 PM

Closed by commit rCOMM29f99ee4dfad: Establish olm notifs session on both sides during registration. · Explain WhyMay 29 2023, 8:41 PM

This revision was automatically updated to reflect the committed changes.

marcin added a commit: rCOMM29f99ee4dfad: Establish olm notifs session on both sides during registration.

ashoat added a reverting change: rCOMM080355ee6040: Revert "Establish olm notifs session on both sides during registration".May 31 2023, 11:51 AM

marcin reopened this revision.May 31 2023, 3:36 PM

This revision is now accepted and ready to land.May 31 2023, 3:36 PM

Rebase before landing

Harbormaster completed remote builds in B19951: Diff 27341.Jun 1 2023, 1:32 PM

Closed by commit rCOMM7cbc1822445f: Establish olm notifs session on both sides during registration. · Explain WhyJun 1 2023, 1:55 PM

This revision was automatically updated to reflect the committed changes.

marcin added a commit: rCOMM7cbc1822445f: Establish olm notifs session on both sides during registration.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

creators/

account-creator.js

18 lines

responders/

user-responders.js

1 line

lib/

types/

account-types.js

1 line

native/

account/

register-panel.react.js

14 lines

Diff 27349

keyserver/src/creators/account-creator.js

Loading...

keyserver/src/responders/user-responders.js

Loading...

lib/types/account-types.js

Loading...

native/account/register-panel.react.js

Loading...