FEATURE: Delay push notifications as we do with emails (PR #11574)

GitHub

Does this mean if a notification is created in Discourse, we might delay or not send out the push notification?

Does this mean if a notification is created in Discourse, we might delay or not send out the push notification?

The idea is that we delay it, and after the delay we check if the user wasn’t on the site during the delay, which means he already saw the discourse notification and don’t need a push one to go for the mobile device.

Ah icic. This is a good change to have but I kind of wished we had some way to centralise the logic of sending out “external” notifications like push and emails.

Ah icic. This is a good change to have but I kind of wished we had some way to centralise the logic of sending out “external” notifications like push and emails.

Yes, that would be cool, but also a larger change.

But I don’t think we should be using a site settings meant for email on push notifications. If the intention is to make push notifications behave like emails, we should centralised the logic and rename the site setting so that it applies to both.

On Mon, 28 Dec 2020 at 23:43, Rafael dos Santos Silva < notifications@github.com> wrote:

Ah icic. This is a good change to have but I kind of wished we had some way to centralise the logic of sending out “external” notifications like push and emails.

Yes, that would be cool, but also a larger change.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/discourse/discourse/pull/11574#issuecomment-751756604, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABBCQ7TK62NDUEYBSLQXCATSXCRR7ANCNFSM4VHO3ALA .

I agree with @tgxworld - it is counter intuitive that the site setting refers to email when it’s not.

so sorry I am closing this, I think we got to do the bigger change here if we are going down this path.

Also we don’t delay this one… discourse/post_alerter.rb at 626b8465baed15799b89135d79c9b8a00eda3bb7 · discourse/discourse · GitHub

We should be consistent here if we are making a change.

Maybe a push_notification_delay site setting. It would be a bit complex cause when it eventually pushed you have to double confirm the user did not already read it in the intervening time.

Probably worth a dev TODO so we can scope it out.