Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add RSS import/export #750

Merged
merged 3 commits into from
Jul 8, 2024
Merged

Add RSS import/export #750

merged 3 commits into from
Jul 8, 2024

Conversation

vagos
Copy link
Contributor

@vagos vagos commented Jun 29, 2024

When exporting to a file ending with .rss, buku will now generate an RSS feed compatible with RSS readers. The feed encodes each bookmark as an entry with <title>, <url>, <conent> and <category> elements.

The feed generated right now is an Atom feed.

Tested both import and export with: buku -e bk.rss and buku -i bk.rss
Tested feed validity using newsboat and opening it in Brave browser.

@vagos
Copy link
Contributor Author

vagos commented Jun 29, 2024

@LeXofLeviafan My bad, something went wrong with the log and had to open a new PR. Incorporated latest comments too.

buku Show resolved Hide resolved
buku Show resolved Hide resolved
@vagos vagos force-pushed the master branch 3 times, most recently from 59e990e to 3608eba Compare July 1, 2024 01:14
tests/test_buku.py Outdated Show resolved Hide resolved
tests/test_buku.py Outdated Show resolved Hide resolved
@vagos vagos force-pushed the master branch 3 times, most recently from d1f5f63 to 2cdda95 Compare July 7, 2024 23:03
@LeXofLeviafan LeXofLeviafan merged commit 6394fb2 into jarun:master Jul 8, 2024
1 check passed
@LeXofLeviafan
Copy link
Collaborator

@vagos Incidentally, I suggest making pull-requests from feature-branches created for that purpose (and only keeping upstream commits in your master in the projects you've forked with the aim to contribute).

This makes it less confusing to manage the remote updates, and allows for multiple feature branches when needed.

@vagos
Copy link
Contributor Author

vagos commented Jul 8, 2024

@LeXofLeviafan Ah, yes. That makes sense.

By the way, thank you very much for the feedback and help getting this PR ready.

Cheers!

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.

2 participants