You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I expected to be able to enable in-app notifications on any posts in the Reader.
Actual behavior
When I try to enable in-app notifications on some posts, the toggle immediately toggles off. I noticed it happening on a blog (internal p2) where I already follow all posts and comments via email.
The app logs show the following:
2022-07-01 12:57:21:712 🔵 Tracked: reader_toggle_comment_notifications <blog_id: 208157483, default_tab_experiment: dashboard, notifications_enabled: 1, source: reader_post_details_comments, subscription_count: 96>
2022-07-01 12:57:22:037 Error updating notification settings for followed conversation: Optional((extension in WordPressKit):__C.ReaderPostServiceRemote.ResponseError.decodingFailed)
2022-07-01 12:57:22:038 Reader Comments: error toggling notification status: Optional((extension in WordPressKit):__C.ReaderPostServiceRemote.ResponseError.decodingFailed))
This issue likely isn't relevant anymore. The bell icon is removed. Similar functionality exists through "Subscribe" button and following the toast's enable button works fine. Pinged @dvdchr just to be safe.
EDIT: I misunderstood the issue before. The bug remains and needs a fix.
The culprit is the remoteReceivesNotifications we receive as response in ReaderPostServiceRemote. It returns false even though the notifications are enabled from the Web. This causes the remoteReceivesNotifications == receiveNotifications comparison to fail as the local value is true
Expected behavior
I expected to be able to enable in-app notifications on any posts in the Reader.
Actual behavior
When I try to enable in-app notifications on some posts, the toggle immediately toggles off. I noticed it happening on a blog (internal p2) where I already follow all posts and comments via email.
The app logs show the following:
Steps to reproduce the behavior
Tested on iPhone 12 Mini, iOS 15.5, Jetpack app 20.0.1
The text was updated successfully, but these errors were encountered: