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

Article status sync bug #708

Closed
brentsimmons opened this issue May 26, 2019 · 4 comments
Closed

Article status sync bug #708

brentsimmons opened this issue May 26, 2019 · 4 comments
Assignees
Milestone

Comments

@brentsimmons
Copy link
Collaborator

I’m not sure what exactly the bug is, so I’ll describe what I see.

I’m running NetNewsWire on two Macs in my office.

On Mac #1, I refresh, then read everything. Then refresh again, and there’s nothing new. Totally fine.

On Mac #2, I refresh, and I get new articles (as expected) that match what’s on Mac #1. However — they’re marked as unread, when I have read them on Mac #1.

I refresh Mac #2 again, and they’re still unread.

Then I login to the Feedbin website — and they are properly marked as read there.

It’s just on Mac #2 where they’re still marked as unread.

@brentsimmons brentsimmons added this to the 5.0 Alpha milestone May 26, 2019
@brentsimmons
Copy link
Collaborator Author

Ignore the links to issues 1 and 2. :) GitHub made those links.

@brentsimmons
Copy link
Collaborator Author

Maybe there’s a conditional GET thing misbehaving? I haven’t dived into this area of the code, so I don’t know what it could be.

When I refresh Mac 2, the Xcode console shows me this:

2019-05-26 16:39:15.336325-0700 NetNewsWire[33609:3382387] [Feedbin] Refreshing articles...
2019-05-26 16:39:15.480284-0700 NetNewsWire[33609:3363928] [Feedbin] Done refreshing articles.
2019-05-26 16:39:15.480318-0700 NetNewsWire[33609:3363928] [Feedbin] Refreshing article statuses...
2019-05-26 16:39:15.793224-0700 NetNewsWire[33609:3363928] [Feedbin] Done refreshing article statuses.
2019-05-26 16:39:15.793264-0700 NetNewsWire[33609:3363928] [Feedbin] Refreshing missing articles...
2019-05-26 16:39:15.795556-0700 NetNewsWire[33609:3363928] [Feedbin] Done refreshing missing articles.

@vincode-io
Copy link
Member

vincode-io commented May 27, 2019

I think I know what happened here. We save unread statuses for articles that we don't have an article for yet, but the processing to mark a status from unread to read depends on an article being present.

Frankly, I don't know how we haven't ran into this problem yet. I've been bouncing back and forth between my phone and desktop and should have triggered it too by now.

I'll have a fix for it in this evening or first thing tomorrow morning.

vincode-io added a commit that referenced this issue May 27, 2019
@vincode-io
Copy link
Member

I was successfully able to recreate this bug.

Steps:

  1. Set refresh feeds to manual
  2. Subscribe to a new feed on the Feedbin site
  3. Wait for a status sync to get the new unread article statuses into NetNewsWire
  4. Mark all as read on the Feedbin site
  5. Wait for a status sync to get the now read articles statuses into NetNewsWire
  6. Do a full article refresh

Before you would have the new feed articles in an unread state. After the fix, they are in the correct read state.

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

No branches or pull requests

2 participants