fix: Filter out markdown for og:description

[Jarsen136]

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.

👇 __ Let's make a quick check before the contribution.

PR Type

• Bugfix
• Feature
• Refactoring

Context

• Closes Filter out markdown for og:description #6346
• Requires deployment <snek/rubick/worker>

before:

after:

Did your issue had any of the "$" label on it?

• My DOT address: Payout

Copilot Summary

🤖 Generated by Copilot at 93ebc72

Added a function to detect markdown in strings and used it to remove markdown from collection descriptions in meta tags. This improves SEO by providing cleaner and more relevant metadata for collections.

🤖 Generated by Copilot at 93ebc72

Sing, O Muse, of the skillful coder who devised
A cunning function, isMarkdownString, to discern
The subtle signs of links and styles that mark the words
Of sages who compose their wisdom in markdown.

[netlify]

✅ Deploy Preview for koda-canary ready!

Name	Link
🔨 Latest commit	58c09f0
🔍 Latest deploy log	https://app.netlify.com/sites/koda-canary/deploys/649d793aea65060008c378b0
😎 Deploy Preview	https://deploy-preview-6348--koda-canary.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[reviewpad]

AI-Generated Summary: This pull request includes changes in two files to filter out markdown from og:description. A new utility function isMarkdownString has been added in 'markdown.ts', which checks if a given text string is in markdown format. This utility function is utilized in 'CollectionBanner.vue' to check the linked collection's metadata description. If the description contains markdown, an empty string is returned; otherwise, the original description is preserved.

[prury]

I've tested on discord and telegram, works fine.
Only concern of my was users using markdown in their collection description, but i browsed trough some and couldn't find any.

[prury]

is it supposed to work for the NFT inside the collection aswell?

[vikiival]

Sir I think this is not correct. I guess @yangwao wanted to strip out markdown stuff like * [] #

...

[codeclimate]

Code Climate has analyzed commit 07f157a and detected 0 issues on this pull request.

View more on Code Climate.

[Jarsen136]

✅ Updated

before

after

[Jarsen136]

is it supposed to work for the NFT inside the collection aswell?

Yes, it should also be the same now.

[prury]

@Jarsen136 using a link that directs to the collection is still not showing the description without the markdown:

if i use the NFT inside the collection it works properly:

[Jarsen136]

@Jarsen136 using a link that directs to the collection is still not showing the description without the markdown:

It should be related to the cache from discord. Pls use telegram to test the url sharing again. @prury

[prury]

@Jarsen136 using a link that directs to the collection is still not showing the description without the markdown:

It should be related to the cache from discord. Pls use telegram to test the url sharing again. @prury

i had tested on telegram aswell:

might also be a cache problem, any other place where i can test it?

[Jarsen136]

@Jarsen136 using a link that directs to the collection is still not showing the description without the markdown:

It should be related to the cache from discord. Pls use telegram to test the url sharing again. @prury

i had tested on telegram aswell:

might also be a cache problem, any other place where i can test it?

Yes. It's cache. You could test with this link to refresh the cache. https://deploy-preview-6348--koda-canary.netlify.app/stmn/collection/30?t=2

[prury]

@Jarsen136 using a link that directs to the collection is still not showing the description without the markdown:

It should be related to the cache from discord. Pls use telegram to test the url sharing again. @prury

i had tested on telegram aswell:
might also be a cache problem, any other place where i can test it?

Yes. It's cache. You could test with this link to refresh the cache. https://deploy-preview-6348--koda-canary.netlify.app/stmn/collection/30?t=2

works fine ty, will just do a general check and i'll change the label

[prury]

oh, it got merged... 😄