Page MenuHomePhabricator
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
Unknown Object (File)
Fri, Nov 8, 10:35 PM
Unknown Object (File)
Fri, Nov 8, 1:40 PM
Unknown Object (File)
Tue, Oct 29, 12:07 AM
Unknown Object (File)
Oct 25 2024, 1:18 AM
Unknown Object (File)
Oct 25 2024, 1:18 AM
Unknown Object (File)
Oct 25 2024, 1:18 AM
Unknown Object (File)
Oct 25 2024, 1:18 AM
Unknown Object (File)
Oct 10 2024, 8:13 PM
View All 60 Files
Subscribers
tomek

Details

Reviewers
ashoat
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
rCOMM Comm
Branch
arcpatch-D11712 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

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
atul added a comment.May 1 2024, 4:30 PM

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

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

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

  1. Ensured that UNSUPPORTED message appeared in web Redux store:

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

atul updated this revision to Diff 39723.May 1 2024, 4:35 PM

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

PathSize
lib/
shared/
5 lines

Diff 39723

View Options

lib/shared/message-utils.js

Loading...