Page MenuHomePhabricator
Differential D11609

[lib] Narrow `TraditionalRelationshipRequest` to exclude 'farcaster' action
ClosedPublic
Actions

Authored by atul on Tue, Apr 9, 9:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 11:13 PM
Unknown Object (File)
Sat, Apr 27, 7:07 AM
Unknown Object (File)
Fri, Apr 26, 9:58 PM
Unknown Object (File)
Fri, Apr 26, 4:21 PM
Unknown Object (File)
Fri, Apr 26, 1:40 PM
Unknown Object (File)
Fri, Apr 26, 9:14 AM
Unknown Object (File)
Mon, Apr 22, 4:50 AM
Unknown Object (File)
Sun, Apr 21, 12:12 PM
View All 16 Files
Subscribers
ashoat

Details

Reviewers
ginsu
tomek
will
varun
Commits
rCOMM1638313148ad: [lib] Narrow `TraditionalRelationshipRequest` to exclude 'farcaster' action
Summary

Narrow TraditionalRelationshipRequest to exclude 'farcaster' action and make necessary changes across native/web client.

Will update validators/keyserver in subsequent diffs.

Depends on D11607

Test Plan

flow + validators will be tested in subsequent diffs.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Tue, Apr 9, 9:41 PM
Herald added a subscriber: ashoat. · View Herald TranscriptTue, Apr 9, 9:41 PM
atul added a child revision: D11610: [keyserver] Narrow `updateRelationshipInputValidator` to exclude `farcaster` action.Tue, Apr 9, 9:53 PM
Harbormaster completed remote builds in B28111: Diff 38967.Tue, Apr 9, 9:57 PM
atul requested review of this revision.Tue, Apr 9, 9:57 PM
tomek accepted this revision.Wed, Apr 10, 6:51 AM
tomek added inline comments.
lib/hooks/relationship-prompt.js
12–13 ↗(On Diff #38967)

Can we merge these imports?

This revision is now accepted and ready to land.Wed, Apr 10, 6:51 AM
atul added inline comments.Wed, Apr 10, 12:20 PM
lib/hooks/relationship-prompt.js
12–13 ↗(On Diff #38967)

Ah yeah, thanks for catching that

atul updated this revision to Diff 39006.Wed, Apr 10, 12:20 PM

address @tomek's feedback

Harbormaster completed remote builds in B28138: Diff 39006.Wed, Apr 10, 12:38 PM
atul added a comment.Wed, Apr 17, 12:13 PM

This diff is safe to land since we're just narrowing flow types on the clients. No runtime changes so this shouldn't break anything.

atul updated this revision to Diff 39196.Wed, Apr 17, 12:15 PM

rebase and land

atul added a comment.Wed, Apr 17, 12:24 PM

JK, we need D11606 to land this

0ed33c.png (1×1 px, 606 KB)

Harbormaster failed remote builds in B28274: Diff 39196!Wed, Apr 17, 12:28 PM
atul updated this revision to Diff 39249.Thu, Apr 18, 1:20 PM

rebase and land

This revision was landed with ongoing or failed builds.Thu, Apr 18, 1:26 PM
Closed by commit rCOMM1638313148ad: [lib] Narrow `TraditionalRelationshipRequest` to exclude 'farcaster' action (authored by atul). · Explain Why
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMM1638313148ad: [lib] Narrow `TraditionalRelationshipRequest` to exclude 'farcaster' action.
Harbormaster completed remote builds in B28308: Diff 39249.Thu, Apr 18, 1:46 PM

Revision Contents
Changeset List

PathSize
lib/
hooks/
6 lines
shared/
4 lines
types/
2 lines
native/
chat/
settings/
4 lines
profile/
6 lines
web/
settings/
relationship/
4 lines

Diff 39255

View Options

lib/hooks/relationship-prompt.js

Loading...
View Options

lib/shared/relationship-utils.js

Loading...
View Options

lib/types/relationship-types.js

Loading...
View Options

native/chat/settings/thread-settings-edit-relationship.react.js

Loading...
View Options

native/profile/relationship-list-item.react.js

Loading...
View Options

web/settings/relationship/add-users-list-modal.react.js

Loading...