mvdicarlo / postybirb-plus

Remake of PostyBirb using React and NestJS
BSD 3-Clause "New" or "Revised" License
100 stars 26 forks source link

Don't prepend space to first appended tag #250

Closed askmeaboutlo0m closed 1 year ago

askmeaboutlo0m commented 1 year ago

Appending tags already puts two newlines at the end of the description, there's no need for another space. I think there was also an off-by-one in the length check here, allowing one less character than the limit, which is now rectified.

This applies to the generic website tag appender and also adds some special handling for Bluesky, which needs to do a special dance to do its character counting. This unifies the logic, getting rid of a pile of duplicate code in the Bluesky integration.

leaftail1880 commented 1 year ago

Finally someone did it. Thanks!