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

Skipped push of article ##### to Apple News because it is already in sync. #1125

Open
alexbrothman opened this issue May 17, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@alexbrothman
Copy link

Description of the bug

I have several articles with a Sync Status of Needs to be updated. When I click Pubish I see an error that says Skipped push of article ##### to Apple News because it is already in sync..

Additionally, when I attempt to delete an article from Apple News using the plugin, I receive a message that the article has been deleted but it is not actually removed from Apple News. I manually deleted a published article from Apple News and attempted re-publish it, though I received the same Skipped push of article ##### to Apple News because it is already in sync. message.

Uploading Screenshot 2024-05-17 at 11.09.58.png…

Steps To Reproduce

  1. From, /wp-admin/admin.php?page=apple_news_index, identify an article with a Sync Status of Needs to be updated.
  2. Click Publish on the article that needs to be updated.

Additional Information

No response

@alexbrothman alexbrothman added the bug Something isn't working label May 17, 2024
@timatron
Copy link

We're seeing this problem pretty often. To reproduce:

  1. publish an article
  2. backdate the article and update it

Article never makes it to apple news and has this same issue.

@kevinfodness can this be solved by having articles only published on an hourly cronjob?

@kevinfodness
Copy link
Member

No, it's an issue with how we're calculating what constitutes a change to the article. It's a subset of fields (title, post content, a few others). This functionality is in need of overhaul so that we regenerate the JSON and compare it to a prior hash rather than checking individual fields (e.g., a post itself may not have changed at all, but a change to the theme would necessitate an update to the article in order to capture the updated theme value).

@julesyoungberg
Copy link

I am getting this after attempting to delete an article from apple news so that we could republish it. The reason for doing so was that it was not appearing in our feed even though it was accessible by link. Looked like the meta was not properly deleted, but after deleting it manually I still get the same error 😕

@timatron
Copy link

timatron commented Jul 1, 2024

We have the same issue here @julesyoungberg

@mogmarsh
Copy link

mogmarsh commented Nov 6, 2024

@kevinfodness I wonder if a good first step would be to add a checkbox for "Force Update" that would just skip this comparison if checked. This would allow users to force an update for any reason they felt necessary, and could be followed up later with the JSON hash comparison.

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

No branches or pull requests

5 participants