Skip to content

Add feeds - #671

Merged
nickserv merged 1 commit into
masterfrom
add-feeds
Nov 10, 2020
Merged

Add feeds#671
nickserv merged 1 commit into
masterfrom
add-feeds

Conversation

@nickserv

@nickserv nickserv commented Nov 9, 2020

Copy link
Copy Markdown
Member

Docusaurus supports RSS and Atom feeds but we haven't enabled them

@nickserv nickserv changed the title Enable all feeds for all posts (fixes #670) Add feeds (fixes #670) Nov 9, 2020
@nickserv nickserv linked an issue Nov 9, 2020 that may be closed by this pull request
@nickserv nickserv changed the title Add feeds (fixes #670) Add feeds Nov 9, 2020
@nickserv
nickserv marked this pull request as ready for review November 9, 2020 09:25
@nickserv

nickserv commented Nov 9, 2020

Copy link
Copy Markdown
Member Author

All the descriptions seem to be truncated to only a few words and I'm not seeing a config option for the feed generation.

<description>
  <![CDATA[ Hello friends! I'm pleased to announce the recent updates to the testing-library ]]>
</description>

Is there a way to disable that?

@MatanBobi

MatanBobi commented Nov 9, 2020

Copy link
Copy Markdown
Member

All the descriptions seem to be truncated to only a few words and I'm not seeing a config option for the feed generation.

<description>
  <![CDATA[ Hello friends! I'm pleased to announce the recent updates to the testing-library ]]>
</description>

Is there a way to disable that?

I think it's possible by adding a truncate comment in the md file that will signal where to truncate the post so everything above that will be the summary :)

<!--truncate-->

@nickserv

nickserv commented Nov 9, 2020

Copy link
Copy Markdown
Member Author

I'm aware, but ideally it would be nice to always ow the full article, especially since we're not worried about monetization or retention

@MatanBobi

Copy link
Copy Markdown
Member

Well, I couldn't find anything on the documentation nor in the code.
Maybe we can suggest them if we find it necessary and I think I'll be able to create a PR for that.

@nickserv

nickserv commented Nov 9, 2020

Copy link
Copy Markdown
Member Author

Yea could be a good thing do check on upstream. Should we do that first or merge for now?

@MatanBobi

Copy link
Copy Markdown
Member

IMO we can merge this for now and I can open an issue there to see if they'd like me to raise a PR to add that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RSS feed for the blog

2 participants