Page MenuHomePhabricator
Differential D11459

[lib] Implement `UpdateRelationshipSpec.shimUnsupportedMessageInfo`
ClosedPublic
Actions

Authored by atul on Mar 28 2024, 12:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 1:52 PM
Unknown Object (File)
Sun, Apr 21, 5:10 PM
Unknown Object (File)
Wed, Apr 17, 12:49 AM
Unknown Object (File)
Sat, Apr 13, 5:29 PM
Unknown Object (File)
Sat, Apr 13, 11:49 AM
Unknown Object (File)
Thu, Apr 11, 4:33 PM
Unknown Object (File)
Thu, Apr 11, 10:38 AM
Unknown Object (File)
Fri, Apr 5, 5:11 AM
View All 13 Files
Subscribers
ashoat

Details

Reviewers
ginsu
tomek
varun
will
ashoat
Commits
rCOMM88077c9da4b0: [lib] Implement `UpdateRelationshipSpec.shimUnsupportedMessageInfo`
Summary

Shim UpdateRelationship messages with farcaster_mutual operation for now.

Based on the following from hackathon branch: https://github.com/CommE2E/comm/commit/6dcee7ca9c467951a11086fce416c85bb7f3d385

Depends on D11458

Test Plan

Unsupported robotext appears as expected:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Mar 28 2024, 12:54 PM
Herald added a subscriber: ashoat. · View Herald TranscriptMar 28 2024, 12:54 PM
Harbormaster completed remote builds in B27799: Diff 38494.Mar 28 2024, 1:10 PM
atul requested review of this revision.Mar 28 2024, 1:10 PM
ashoat accepted this revision.Mar 28 2024, 7:09 PM

I previously reviewed this code in person

lib/shared/messages/update-relationship-message-spec.js
241 ↗(On Diff #38494)

Can we avoid copy-pasting this?

Line 196 can be:

ET`${string}`
This revision is now accepted and ready to land.Mar 28 2024, 7:09 PM
atul updated this revision to Diff 38878.Fri, Apr 5, 5:45 PM

address feedback + land

Harbormaster completed remote builds in B28032: Diff 38878.Fri, Apr 5, 6:01 PM
atul updated this revision to Diff 38894.Mon, Apr 8, 6:16 AM

land

This revision was landed with ongoing or failed builds.Mon, Apr 8, 6:17 AM
Closed by commit rCOMM88077c9da4b0: [lib] Implement `UpdateRelationshipSpec.shimUnsupportedMessageInfo` (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM88077c9da4b0: [lib] Implement `UpdateRelationshipSpec.shimUnsupportedMessageInfo`.
Harbormaster completed remote builds in B28045: Diff 38894.Mon, Apr 8, 6:32 AM

Revision Contents
Changeset List

PathSize
lib/
shared/
messages/
49 lines

Diff 38895

View Options

lib/shared/messages/update-relationship-message-spec.js

Loading...