Paths

Table of Contentst

Differential D11712

[lib] Shim unsupported messages for `web`/`desktop`
ClosedPublic
Actions

Authored by atul on Apr 22 2024, 10:13 AM.

Tags

None

Referenced Files

	F3388259: D11712.id39724.diff
	Fri, Nov 29, 1:25 PM

	F3387995: D11712.diff
	Fri, Nov 29, 12:13 PM

	Unknown Object (File)
	Tue, Nov 26, 3:33 AM

	Unknown Object (File)
	Fri, Nov 8, 10:35 PM

	Unknown Object (File)
	Fri, Nov 8, 1:40 PM

	Unknown Object (File)
	Oct 29 2024, 12:07 AM

	Unknown Object (File)
	Oct 25 2024, 1:18 AM

	Unknown Object (File)
	Oct 25 2024, 1:18 AM

View All 63 Files

Subscribers

Details

Reviewers

Commits

rCOMM4c5a86dbda62: [lib] Shim unsupported messages for `web`/`desktop`

Summary

We currently aren't shimming messages for web/desktop, probably because we weren't previously persisting things on those platforms?

However, now that we are persisting messages we'll want to shim/unshim messages. Specically we want to shim/unshim the new UPDATE_RELATIONSHIP message type.

Test Plan

WARNING: I'm still reading through codebase to make sure that this change is safe and can be made as-is. I also haven't yet tested this function with UPDATE_RELATIONSHIP messages to make sure it behaves as expected. Just putting this up to reference in comment on D11707

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Apr 22 2024, 10:13 AM

Herald added a subscriber: tomek. · View Herald TranscriptApr 22 2024, 10:13 AM

atul mentioned this in D11707: [keyserver] Send farcaster_mutual UPDATE_RELATIONSHIP message when establishing Farcaster relationship.Apr 22 2024, 10:16 AM

Harbormaster completed remote builds in B28355: Diff 39344.Apr 22 2024, 10:31 AM

atul requested review of this revision.Apr 22 2024, 10:31 AM

ashoat accepted this revision.Apr 22 2024, 11:01 AM

This revision is now accepted and ready to land.Apr 22 2024, 11:01 AM

ashoat added a child revision: D11707: [keyserver] Send farcaster_mutual UPDATE_RELATIONSHIP message when establishing Farcaster relationship.Apr 26 2024, 5:41 PM

Did testing to ensure UNSUPPORTED messages appear as expected on web client:

Rewrote the request in updateRelationships so that I could avoid making client changes:

c27e4f.png (740×1 px, 112 KB)

Ensured that UNSUPPORTED message appeared in web Redux store:

870d7f.png (304×972 px, 51 KB)

rebase and land

Harbormaster completed remote builds in B28617: Diff 39723.May 1 2024, 4:52 PM

Closed by commit rCOMM4c5a86dbda62: [lib] Shim unsupported messages for `web`/`desktop` (authored by atul). · Explain WhyMay 1 2024, 4:54 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM4c5a86dbda62: [lib] Shim unsupported messages for `web`/`desktop`.

Revision Contents
Changeset List

Path

Size

lib/

shared/

message-utils.js

5 lines

Diff 39724

lib/shared/message-utils.js

Loading...