well, it doesn't work with the public instance, but that's probably to be expected (considering the 1000+ minutes it says it takes to fetch users)?

i'll try with a self-hosted version sometime later

On 12/8/2023 6:46 PM, ~jofkos wrote:

#3 and #13 are (pretty much) fixed. Would be interesting to know whether it works for Akkoma, too, now.

#3 and #13 are (pretty much) fixed. Would be interesting to know whether it works for Akkoma, too, now.

Amazing, happy to hear that it actually worked! :)

I'm going to close this todo since ~jofkos made GoToSocial work!

There is already a todo for Firefish https://todo.sr.ht/~cloutier/bird.makeup/13 if we can continue the conversation there, we would avoid sending notification to the GoToSocial users following this issue :)

As I have debugging already set up, I just had a look: Turns out, bird.makeup fails to deserialize activities originating from Firefish because it assumes the @context is a string – which it isn't. This patchset should fix it: :)

https://git.sr.ht/~jofkos/bird.makeup/commit/948ad963a048d961b9f897f0e33b76b8a261cfee

Follow-acknowledgements should be sent (part of the problem with GoToSocial was, that those had the wrong Content-Type), so if you have any more information about what is going wrong (like an error message or something similar) I'd be happy to have a look!

Thanks! One thing I just noticed in the code I submitted, though, is that DeleteRequestedAsync will likely fail right now. GetUser requires a local user (to specify a location for the keyid) and I wasn't sure where to get that from, so I had just put null as a placeholder for now. Just a heads-up!

Also, I was indeed able to follow bird.makeup users from my GoToSocial. :)

Thank you for your contribution ~jofkos! I have pulled your changes and deployed them to production. Any GoToSocial user here can confirm if it works now?

No worries, same here. I wasn't even sure where to send the patchset to, so I hope it reached ~cloutier.

No, I just submitted it!