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

"Mark All as Read" doesn't mark articles as read on the server #561

Closed
dontobi opened this issue Jan 23, 2024 · 8 comments · Fixed by #582
Closed

"Mark All as Read" doesn't mark articles as read on the server #561

dontobi opened this issue Jan 23, 2024 · 8 comments · Fixed by #582
Assignees
Labels
bug Something isn't working
Milestone

Comments

@dontobi
Copy link
Contributor

dontobi commented Jan 23, 2024

1. Environment

  • Device: Pixel 8
  • OS: Android 14
  • APP: 0.9.11 (latest Actions Release)

2. Describe the bug

I've been using ReadYou for a few days with a FreshRSS instance that I run via Docker on my own server. With a version from yesterday I was not able to mark all articles as read. With the latest version of Actions this does not work at the moment.

@dontobi dontobi added the bug Something isn't working label Jan 23, 2024
@mbestavros
Copy link
Collaborator

Can confirm, seeing the same bug.

I'm also seeing it on the Fever API, as well, which is a regression. It was working fine before.

@mbestavros
Copy link
Collaborator

@Ashinch After testing some of the Github builds, this bug appears to have been introduced with the Google Reader API commit: fa2787b

Hard to pin down any closer than that, given it's such a big commit 😅

@mbestavros mbestavros changed the title Mark as read without function GReader: "Mark as read" doesn't mark articles as read on the server Jan 23, 2024
@mbestavros mbestavros changed the title GReader: "Mark as read" doesn't mark articles as read on the server GReader: "Mark All as Read" doesn't mark articles as read on the server Jan 23, 2024
@mbestavros mbestavros changed the title GReader: "Mark All as Read" doesn't mark articles as read on the server "Mark All as Read" doesn't mark articles as read on the server Jan 23, 2024
@Ashinch Ashinch self-assigned this Jan 24, 2024
@dontobi
Copy link
Contributor Author

dontobi commented Jan 28, 2024

Has the problem already been resolved? I have seen the commits 01f5f38 and 9b061e4.

Edit: The bug has not yet been fixed. I just ran the test with the build b82ad59

So I went back to build bb978c8, which works for me without any problems.

@Ashinch
Copy link
Owner

Ashinch commented Jan 28, 2024

@dontobi It has not been tested and may not have been fully solved.

@Ashinch Ashinch added this to the 0.9.12 milestone Jan 30, 2024
@dontobi
Copy link
Contributor Author

dontobi commented Jan 31, 2024

If you want me to test a build, please let me know.

@Ashinch
Copy link
Owner

Ashinch commented Feb 1, 2024

If you want me to test a build, please let me know.

Hi, @dontobi

I will commit the relevant fixes to the main branch in the next few days. If you want to test it, you can get the nightly build from GitHub Actions at that time.

@SoiledBrush
Copy link

Still encountering this issue on fever, latest build.

@Ashinch
Copy link
Owner

Ashinch commented Feb 9, 2024

Still encountering this issue on fever, latest build.

@SoiledBrush

Oh, that looks bad. Please open a separate issue specifically for mark all as read via Fever API. I’ll follow up on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants