alleyinteractive / apple-news

The Publish to Apple News plugin enables your WordPress blog content to be published to your Apple News channel.
https://wordpress.org/plugins/publish-to-apple-news/
GNU General Public License v3.0
155 stars 71 forks source link

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

Open alexbrothman opened 6 months ago

alexbrothman commented 6 months ago

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

timatron commented 5 months ago

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 commented 5 months ago

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 commented 4 months ago

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 commented 4 months ago

We have the same issue here @julesyoungberg

mogmarsh commented 1 week ago

@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.