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
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 39344

View Options

lib/shared/message-utils.js

Loading...