Page MenuHomePhabricator
Differential D13779

reset avatar when deleteFarcasterChannelTag is called
Needs RevisionPublic
Actions

Authored by varun on Wed, Oct 23, 10:50 AM.
Tags
None
Referenced Files
F3061296: D13779.diff
Thu, Oct 24, 12:25 PM
F3061258: D13779.id45360.diff
Thu, Oct 24, 12:25 PM
F3058667: D13779.id45361.diff
Thu, Oct 24, 1:04 AM
F3058027: D13779.id45361.diff
Wed, Oct 23, 10:51 PM
F3058026: D13779.id45358.diff
Wed, Oct 23, 10:51 PM
F3058025: D13779.id45352.diff
Wed, Oct 23, 10:51 PM
F3058024: D13779.id45360.diff
Wed, Oct 23, 10:51 PM
F3058017: D13779.id.diff
Wed, Oct 23, 10:51 PM
View All 18 Files
Subscribers
tomek

Details

Reviewers
ashoat
Summary

if a tagged community has a farcaster avatar, we should reset it to NULL when the tag is removed

Depends on D13778

Test Plan

Tested in conjunction with previous diff

idnameavatardescription

Created community

84426Oct23NULLNULL

Tagged community with spam channel. NULL description was replaced with description from neynar. (also confirmed that avatar and description were updated on client)

84426Oct23{"type":"farcaster"}Dedicated channel for everything $SPAM

Removed tag. avatar was reset to NULL. description remained

84426Oct23NULLDedicated channel for everything $SPAM

Removed description from native app. It was set to an empty string

84426Oct23NULL

Added a new description from app.

84426Oct23NULLAsdf

Tagged community with a different channel. Avatar changed but description remained the same.

84426Oct23{"type":"farcaster"}Asdf

Removed tag

84426Oct23NULLAsdf

Removed description

84426Oct23NULL

Tagged community with a different channel. empty string description was replaced with description from neynar

84426Oct23{"type":"farcaster"}your musings, thoughts & dreams; welcome here

Removed tag and set a custom emoji avatar

84426Oct23{"type":"emoji","emoji":"?","color":"57697f"}your musings, thoughts & dreams; welcome here

Added and removed a new tag, avatar was not overwritten

84426Oct23{"type":"emoji","emoji":"?","color":"57697f"}your musings, thoughts & dreams; welcome here

Diff Detail

Repository
rCOMM Comm
Branch
farcaster-thread-avatars (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun created this revision.Wed, Oct 23, 10:50 AM
Herald added a subscriber: tomek. · View Herald TranscriptWed, Oct 23, 10:50 AM
varun edited the test plan for this revision. (Show Details)Wed, Oct 23, 10:55 AM
varun published this revision for review.Wed, Oct 23, 11:00 AM

planning changes to rethink this a little

varun planned changes to this revision.Wed, Oct 23, 11:00 AM
Harbormaster completed remote builds in B32350: Diff 45352.Wed, Oct 23, 12:52 PM
varun updated this revision to Diff 45358.Wed, Oct 23, 7:01 PM

use updateThread in the background

varun updated this revision to Diff 45360.Wed, Oct 23, 7:09 PM

fix spacing

varun updated this revision to Diff 45361.Wed, Oct 23, 7:10 PM

undo last change

ashoat requested changes to this revision.Wed, Oct 23, 7:10 PM
ashoat added inline comments.
keyserver/src/deleters/farcaster-channel-tag-deleters.js
74

Unsafe if we can't find the community

79

Result should be returned to clients and reduced

This revision now requires changes to proceed.Wed, Oct 23, 7:10 PM
Harbormaster completed remote builds in B32353: Diff 45358.Wed, Oct 23, 7:20 PM
Harbormaster completed remote builds in B32355: Diff 45360.Wed, Oct 23, 7:35 PM
Harbormaster completed remote builds in B32356: Diff 45361.Wed, Oct 23, 7:43 PM

Revision Contents
Changeset List

PathSize
keyserver/
src/
deleters/
23 lines

Diff 45360

View Options

keyserver/src/deleters/farcaster-channel-tag-deleters.js

Loading...