askvortsov1 / flarum-pwa

Progressive Web App support for Flarum
MIT License
34 stars 12 forks source link

The content of a new post notification should include the content of this post, not the content of the first post in the discussion #36

Closed rafaucau closed 2 years ago

rafaucau commented 2 years ago

The content of a new post notification should include the content of this post, not the content of the first post in the discussion. It would be more useful for users to have the content of the post they are notified about there.

askvortsov1 commented 2 years ago

For newPost notifications from the subscriptions extension, this should already be the case. Which type of notification are you seeing issues with?

rafaucau commented 2 years ago

@askvortsov1 Ah, indeed. The problem is with push notifications from fof/byobu.

askvortsov1 commented 2 years ago

Should be fixed in v3.1.3

rafaucau commented 2 years ago

Thanks!