Page MenuHomePhabricator
Differential D11458

[lib] Handle `farcaster_mutual` case in `robotext/notificationTexts`
ClosedPublic
Actions

Authored by atul on Mar 28 2024, 12:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 25, 2:07 AM
Unknown Object (File)
Thu, Nov 21, 2:46 AM
Unknown Object (File)
Thu, Nov 21, 2:46 AM
Unknown Object (File)
Thu, Nov 7, 12:15 PM
Unknown Object (File)
Thu, Nov 7, 5:47 AM
Unknown Object (File)
Fri, Nov 1, 5:47 PM
Unknown Object (File)
Wed, Oct 30, 5:41 AM
Unknown Object (File)
Oct 24 2024, 11:26 AM
View All 93 Files
Subscribers
ashoat

Details

Reviewers
ashoat
ginsu
varun
tomek
will
Commits
rCOMM444cce2542c3: [lib] Handle `farcaster_mutual` case in `robotext/notificationTexts`
Summary

Show different message in robotext/notificationTexts when operation is of type farcaster_mutual.

Based on https://github.com/CommE2E/comm/commit/7d43cd1575df7b6f0a23d0e5dd88d63bbfab7bf8

(adding @ashoat as blocking to get sign off on copy)

Depends on D11457

Test Plan

Appears as expected:

3be31b.png (2×1 px, 736 KB)

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul created this revision.Mar 28 2024, 12:11 PM
atul published this revision for review.Mar 28 2024, 12:12 PM
atul edited the test plan for this revision. (Show Details)
Harbormaster completed remote builds in B27797: Diff 38492.Mar 28 2024, 12:39 PM
atul added a child revision: D11459: [lib] Implement `UpdateRelationshipSpec.shimUnsupportedMessageInfo`.Mar 28 2024, 12:54 PM
ashoat accepted this revision.Mar 28 2024, 4:30 PM

We have ENG-7514 to track improving the robotext here, so that it mentions the user's Farcaster username

If we add NeynarClient to keyserver, then we can do it for notifs too

lib/shared/messages/update-relationship-message-spec.js
226–227

I wonder if we should add an invariant here for safety

This revision is now accepted and ready to land.Mar 28 2024, 4:30 PM
atul updated this revision to Diff 38876.Apr 5 2024, 4:38 PM

address feedback

This revision was landed with ongoing or failed builds.Apr 5 2024, 4:41 PM
Closed by commit rCOMM444cce2542c3: [lib] Handle `farcaster_mutual` case in `robotext/notificationTexts` (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM444cce2542c3: [lib] Handle `farcaster_mutual` case in `robotext/notificationTexts`.
Harbormaster completed remote builds in B28031: Diff 38876.Apr 5 2024, 4:54 PM

Revision Contents
Changeset List

PathSize
lib/
shared/
messages/
16 lines

Diff 38492

View Options

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

Loading...