feat: add full-text output to RSS feed

EveSunMaple / Frosti

A clean, elegant, and fast static blog template! 🚀 Developed with Astro

https://frosti.saroprock.com

GNU General Public License v3.0

200 stars 31 forks source link

feat: add full-text output to RSS feed #51

Closed yaotutu closed 1 week ago

yaotutu commented 1 week ago

This pull request introduces a new feature to control whether the RSS feed includes full text or not. The most important changes include the addition of a new constant for this feature and modifications to the RSS feed generation to use this constant.

New feature addition:

src/consts.ts: Added a new constant RSS_FULL_TEXT to control whether the RSS feed includes full text.

RSS feed generation updates:

src/pages/rss.xml.js: Imported the new RSS_FULL_TEXT constant and the marked library. Updated the RSS feed generation logic to include full text in the feed items if RSS_FULL_TEXT is set to true.consts file to toggle the feature

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
frosti	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 19, 2024 7:21am

sonarcloud[bot] commented 1 week ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

EveSunMaple commented 1 week ago

I had tested this solution earlier, but it caused issues in .mdx files. For instance, it retained the import statements and failed to correctly parse components, which is why I didn't incorporate it right away.

This is indeed a tricky matter, as the Astro documentation explicitly mentions this limitation. Not only that, even plain .md files can have issues—some HTML examples might reference local paths (e.g., /image/r.png), which may not be accessible in RSS parsers.

Please allow me to conduct further tests myself before considering this PR.