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

Update RSS Reader extension - remember story last read + filter by read status (close issue) (re-open PR) #16655

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

xmok
Copy link
Contributor

@xmok xmok commented Jan 27, 2025

NOTE:

Re-opened the PR since it was stuck for some reason. (previous PR: #16654)

PR-stuck

Description

KNOWN LIMITATION: need to re-load or refresh the View to update the lastRead. We could improve it through some form of mutation and/or store all stories locally or something new but that will extend scope of this PR - will be visited in future.

Screencast

rss-reader-remember-last-read.mp4

Checklist

@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: rss-reader Issues related to the rss-reader extension OP is contributor The OP of the PR is a contributor of the extension labels Jan 27, 2025
@raycastbot
Copy link
Collaborator

Thank you for your contribution! 🎉

🔔 @fuksman @pernielsentikaer @pomdtr you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

You can expect an initial review within five business days.

@pernielsentikaer
Copy link
Collaborator

Thanks for the fix @xmok

@fuksman do you want to check this?

@pernielsentikaer pernielsentikaer self-assigned this Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension fix / improvement Label for PRs with extension's fix improvements extension: rss-reader Issues related to the rss-reader extension OP is contributor The OP of the PR is a contributor of the extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[RSS Reader] Add an option to hide read/viewed articles in “Read All Stories”
3 participants