💾 Archived View for bbs.geminispace.org › s › meta › 19154 captured on 2024-08-31 at 15:42:34. Gemini links have been rewritten to link to archived content
-=-=-=-=-=-=-
BBS has been updated to a work-in-progress version of Bubble.
There are a couple of useful additions so far:
In addition to the comment author, moderators and the admin have the ability to merge comments. I'll be applying this in cases where the same user comments multiple times in a row.
Aug 21 · 10 days ago · 👍 bubstance, Queen_City_Nerd, alice-sur-le-nuage, Allscop
🎵 alice-sur-le-nuage · Aug 21 at 17:46:
Thanks !
🚲 CitySlicker · Aug 21 at 22:00:
comment action that merges the comment into the previous post in the thread, if the authors are the same
Thanks for this. I think will help posts sorted under the “hotness” algorithm. So you can’t just bump your own post to the top
🕹️ skyjake [OP/mod...] · Aug 22 at 03:30:
@CitySlicker The hotness ranking already does take that into account. It's the number of individual people commenting instead of the comments that counts.
In addition to keeping things tidier, the merging action will help write long comments easier since one does not have to go into composer mode for that.
@skyjake Nice to see the edit parameter here. I tried it in my koreader client, but get a bubble error that NoneType has no parameter startswith. Any idea what that could mean?
🕹️ skyjake [OP/mod...] · Aug 25 at 05:04:
Looks like my code assumes the Titan "mime" parameter is always included in the request, even though it should be optional. The code is trying to check that the MIME type of the edited content is text.
I guess your client doesn't send "mime"? I'll update Bubble to assume it's text if unspecified, maybe just detecting valid UTF-8 encoding.
EDIT: Fix applied.
@skyjake Thanks, it seems that is the problem -- my client omits the mime param if it's text/gemini, and if I set it to text/plain instead then editing works. However, I'm using the present tense because I still get the same error on editing with no mime parameter.
🕹️ skyjake [OP/mod...] · Aug 25 at 16:35:
Thanks for testing. I applied another fix. I missed a second usage of the mime parameter... At least now when I modify Lagrange to omit "mime", it works for me. (Should've tested properly the first time, sorry.)
@skyjake Thanks, now it works! Much better UI than the old approach.
🕹️ skyjake [OP/mod...] · Aug 27 at 12:03:
Great!
Yeah I agree, the edit actions are perfectly suited for this use case. :)