notedeck icon indicating copy to clipboard operation
notedeck copied to clipboard

235 follow unfollow user

Open kuba-04 opened this issue 7 months ago • 5 comments

Closes #235

kuba-04 avatar Apr 05 '25 14:04 kuba-04

hey guys, while doing this I see such errors when the note is submitted:

image

I was thinking I might have a look, but since the follow/unfollow works (I am checking on my testing account) I thought it could be something unrelated to this feature, but I don't know. I thought I better ask you first, maybe you have seen this before and have some idea!

kuba-04 avatar Apr 05 '25 14:04 kuba-04

taking a look at this now!

jb55 avatar Apr 15 '25 15:04 jb55

there are lots of issues with this. it looks like the kind3 creation is destructive which is not good. it shouldn't be removing existing tags. what we probably want is an API for creating a note builder from an existing note.

I will iterate on this.

jb55 avatar Apr 15 '25 17:04 jb55

rebased and I'm iterating on this on the follow branch:

https://github.com/damus-io/notedeck/compare/follow

jb55 avatar Apr 15 '25 17:04 jb55

alright! Good point with Unknown variant. About removing the tags, now I see it, I was following the spec and didn't even think there could be other tags in kind 3 that we might want to copy. I will follow that branch to learn something from it! I will close this PR then

kuba-04 avatar Apr 16 '25 08:04 kuba-04

  • superseded by #927

thanks for getting the ball rolling!

jb55 avatar Jul 16 '25 20:07 jb55

oh great to see this! Closing

kuba-04 avatar Jul 21 '25 15:07 kuba-04