Differential D13752

[web] update EditThreadAvatarMenu to add farcaster option
ClosedPublic
Actions

Authored by varun on Oct 19 2024, 11:25 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Nov 20, 8:17 AM

	Unknown Object (File)
	Wed, Nov 20, 8:16 AM

	Unknown Object (File)
	Wed, Nov 20, 8:16 AM

	Unknown Object (File)
	Mon, Nov 11, 1:21 AM

	Unknown Object (File)
	Thu, Nov 7, 7:49 PM

	Unknown Object (File)
	Thu, Nov 7, 3:09 PM

	Unknown Object (File)
	Thu, Nov 7, 1:22 PM

	Unknown Object (File)
	Wed, Nov 6, 11:05 PM

View All 34 Files

Subscribers

Details

Reviewers

Commits

rCOMM0f9633f9bd67: [web] update EditThreadAvatarMenu to add farcaster option

Summary

this lets a user set a farcaster thread avatar if the thread belongs to a community tagged with a farcaster channel

Depends on D13749

Test Plan

successfully set a farcaster avatar in community root thread and child threads. was able to unset the avatar as well.

menu icon looks the same as in the user avatar menu

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Oct 19 2024, 11:25 AM

Herald added a subscriber: tomek. · View Herald TranscriptOct 19 2024, 11:25 AM

Harbormaster completed remote builds in B32308: Diff 45282.Oct 19 2024, 11:43 AM

varun requested review of this revision.Oct 19 2024, 11:43 AM

varun added a child revision: D13753: [native] let user select farcaster thread avatar.Oct 19 2024, 12:30 PM

varun added inline comments.Oct 19 2024, 12:32 PM

web/avatars/edit-thread-avatar-menu.react.js
143 ↗	(On Diff #45282)	one thing worth noting is that we'll display this menu item even if the thread has inherited a farcaster avatar from its parent as the default

ashoat accepted this revision.Oct 20 2024, 11:44 AM

This revision is now accepted and ready to land.Oct 20 2024, 11:44 AM

Closed by commit rCOMM0f9633f9bd67: [web] update EditThreadAvatarMenu to add farcaster option (authored by varun). · Explain WhyTue, Oct 22, 9:52 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM0f9633f9bd67: [web] update EditThreadAvatarMenu to add farcaster option.

Revision Contents
Changeset List

Path

Size

web/

avatars/

edit-thread-avatar-menu.react.js

46 lines

Diff 45341

web/avatars/edit-thread-avatar-menu.react.js

Loading...