Skip to content

235 follow unfollow user #799

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kuba-04
Copy link

@kuba-04 kuba-04 commented Apr 5, 2025

Closes #235

kuba-04 added 2 commits April 3, 2025 18:11
Signed-off-by: Jakub Gladysz <[email protected]>
Signed-off-by: Jakub Gladysz <[email protected]>
@kuba-04
Copy link
Author

kuba-04 commented Apr 5, 2025

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!

@jb55
Copy link
Contributor

jb55 commented Apr 15, 2025

taking a look at this now!

@jb55
Copy link
Contributor

jb55 commented Apr 15, 2025

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
Copy link
Contributor

jb55 commented Apr 15, 2025

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

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

@kuba-04
Copy link
Author

kuba-04 commented Apr 16, 2025

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Follow and unfollow user
2 participants