Paths

Table of Contentst

Differential D13271

Fix input types from D13050
ClosedPublic
Actions

Authored by marcin on Mon, Sep 9, 6:23 AM.

Tags

None

Referenced Files

	F2730268: D13271.id44011.diff
	Tue, Sep 17, 7:28 AM

	F2728588: D13271.id44040.diff
	Tue, Sep 17, 5:09 AM

	F2728267: D13271.id44013.diff
	Tue, Sep 17, 4:38 AM

	F2727616: D13271.id43988.diff
	Tue, Sep 17, 4:04 AM

	F2726536: D13271.id43991.diff
	Tue, Sep 17, 2:11 AM

	F2725029: D13271.id43989.diff
	Mon, Sep 16, 11:53 PM

	F2722953: D13271.id44037.diff
	Mon, Sep 16, 7:21 PM

	Unknown Object (File)
	Mon, Sep 16, 6:57 AM

View All 26 Files

Subscribers

Details

Reviewers

Commits

rCOMM0e2412f00c19: Fix input types from D13050

Summary

This differential fixes types according to Ashoat suggestion: https://phab.comm.dev/D13050#inline-76348

Test Plan

flow

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

marcin created this revision.Mon, Sep 9, 6:23 AM

Herald added a subscriber: tomek. · View Herald TranscriptMon, Sep 9, 6:23 AM

marcin mentioned this in D13050: Extract logic of pushInfo -> usersToCollapsableNotifInfo to lib and use for client-based notifs.Mon, Sep 9, 6:33 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Mon, Sep 9, 6:40 AM

Harbormaster failed remote builds in B31495: Diff 43988!

Fix flow in the keyserver

Fix flow

Harbormaster completed remote builds in B31496: Diff 43989.Mon, Sep 9, 7:14 AM

Harbormaster completed remote builds in B31497: Diff 43990.Mon, Sep 9, 7:22 AM

marcin requested review of this revision.Mon, Sep 9, 7:22 AM

Thanks for doing this! Please apply the inline changes before landing, or otherwise re-request review if it introduces Flow errors

lib/push/send-utils.js
97 ↗	(On Diff #43990)
307 ↗	(On Diff #43990)
353 ↗	(On Diff #43990)

This revision is now accepted and ready to land.Mon, Sep 9, 1:42 PM

Update types

I applied changes from latest @ashoat suggestion but it forced me to change mergeUserToCollapsableInfo to perform deeper copy that initial single spread. Therefore re-requesting review.

Harbormaster completed remote builds in B31510: Diff 44011.Tue, Sep 10, 9:55 AM

Thanks! I considered keeping the inputs to mergeUserToCollapsableInfo mutable, but I wasn't able to quickly confirm that it was safe... so I think this is the best bet

This revision is now accepted and ready to land.Tue, Sep 10, 12:31 PM

Rebase before landing

This revision was landed with ongoing or failed builds.Wed, Sep 11, 5:17 AM

Closed by commit rCOMM0e2412f00c19: Fix input types from D13050 (authored by marcin). · Explain Why

This revision was automatically updated to reflect the committed changes.

marcin added a commit: rCOMM0e2412f00c19: Fix input types from D13050.

Harbormaster completed remote builds in B31531: Diff 44037.Wed, Sep 11, 5:26 AM

Revision Contents
Changeset List

Path

Size

keyserver/

src/

push/

4 lines

lib/

push/

26 lines

Diff 44038

keyserver/src/push/send.js

Loading...

lib/push/send-utils.js

Loading...