search

TASVideos / tasvideos

The code for the live TASVideos website
https://tasvideos.org/
GNU General Public License v3.0
62 stars 29 forks source link

Use markdown for URLs in Discord publications #1684

Closed qixils closed 1 year ago

qixils commented 1 year ago

Future work could expand on this with more URLs (say, linking to the responsible users) but figured this might be a good start to reduce visual clutter in the Discord update channels. Not sure if this might be considered too platform-specific to embed as a variable in Post? I figure markdown is relatively platform-agnostic so it's not too bad but 🤷🏻‍♀️

edit: closes #1681

Randomno commented 1 year ago

Would this close #1681?

qixils commented 1 year ago

Looks like it should close #1681 (but not #1353), I'll note it in the body, thanks.

Masterjun3 commented 1 year ago

This looks like a massive amount of duplications. I'm personally not sure whether a Discord specific feature makes sense to be added to something generic as our MediaPublishing system. Wouldn't it have made sense instead of duplicating and adjusting, to just make the formatting generic, and automatically convert it into fallback text for those who don't support it?

I'd say we should either go full generic, or full separation. This PR feels like a difficult to maintain middleground.