Page MenuHomePhabricator
Differential D11609

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

Authored by atul on Apr 9 2024, 9:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 27, 2:07 AM
Unknown Object (File)
Fri, Dec 27, 1:47 AM
Unknown Object (File)
Thu, Dec 26, 10:45 PM
Unknown Object (File)
Fri, Dec 6, 2:55 AM
Unknown Object (File)
Nov 28 2024, 1:16 AM
Unknown Object (File)
Nov 28 2024, 12:57 AM
Unknown Object (File)
Nov 27 2024, 10:00 PM
Unknown Object (File)
Nov 24 2024, 9:45 AM
View All 79 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
Branch
arcpatch-D11609 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul created this revision.Apr 9 2024, 9:41 PM
Herald added a subscriber: ashoat. · View Herald TranscriptApr 9 2024, 9:41 PM
atul added a child revision: D11610: [keyserver] Narrow `updateRelationshipInputValidator` to exclude `farcaster` action.Apr 9 2024, 9:53 PM
Harbormaster completed remote builds in B28111: Diff 38967.Apr 9 2024, 9:57 PM
atul requested review of this revision.Apr 9 2024, 9:57 PM
tomek accepted this revision.Apr 10 2024, 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.Apr 10 2024, 6:51 AM
atul added inline comments.Apr 10 2024, 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.Apr 10 2024, 12:20 PM

address @tomek's feedback

Harbormaster completed remote builds in B28138: Diff 39006.Apr 10 2024, 12:38 PM
atul added a comment.Apr 17 2024, 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.Apr 17 2024, 12:15 PM

rebase and land

atul added a comment.Apr 17 2024, 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!Apr 17 2024, 12:28 PM
atul updated this revision to Diff 39249.Apr 18 2024, 1:20 PM

rebase and land

This revision was landed with ongoing or failed builds.Apr 18 2024, 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.Apr 18 2024, 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 39196

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