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

[rss] Display RSS Feeds in Notifications #392

Merged
merged 1 commit into from
Jul 12, 2022

Conversation

ricoberger
Copy link
Member

It is now possible to use the RSS plugin to display the items from an
RSS feed within the notifications. RSS feed items are always marked as
unread and info.

We also added a page implementation for the plugin to show the items of
a single RSS feed. This allows us to link from a notification entry to
the complete rss feed.

It is now possible to use the RSS plugin to display the items from an
RSS feed within the notifications. RSS feed items are always marked as
unread and info.

We also added a page implementation for the plugin to show the items of
a single RSS feed. This allows us to link from a notification entry to
the complete rss feed.
@ricoberger ricoberger force-pushed the rss-add-page-and-notifications branch from 94b9d66 to 9afed34 Compare July 12, 2022 19:00
@codecov-commenter
Copy link

Codecov Report

Merging #392 (9afed34) into main (184289e) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #392      +/-   ##
==========================================
+ Coverage   45.20%   45.21%   +0.01%     
==========================================
  Files         134      134              
  Lines       10377    10379       +2     
==========================================
+ Hits         4691     4693       +2     
  Misses       5608     5608              
  Partials       78       78              
Impacted Files Coverage Δ
plugins/plugin-rss/cmd/rss.go 100.00% <100.00%> (ø)
plugins/plugin-rss/pkg/feed/feed.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 184289e...9afed34. Read the comment docs.

@ricoberger ricoberger merged commit bc8ef72 into main Jul 12, 2022
@ricoberger ricoberger deleted the rss-add-page-and-notifications branch July 12, 2022 19:16
@ricoberger ricoberger linked an issue Jul 12, 2022 that may be closed by this pull request
@ricoberger ricoberger added the changelog: added A new feature was added label Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: added A new feature was added
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[app] Add Notifications
2 participants