feat: add push notification when SLA missed

[vishnu-narayanan]

Pull Request Template

Description

• Add in-app notification when SLA target is missed for a conversation
• The notification is send to all users part of the conversation inbox
• Notification is sent to all conversation participants, admins and assignee if present

Fixes https://linear.app/chatwoot/issue/CW-3098/in-app-notification-if-sla-is-missed

Type of change

Please delete options that are not relevant.

• New feature (non-breaking change which adds functionality)

How Has This Been Tested?

• Tested locally by creating a conversation and waiting for the SLA target to miss
• This created an in-app notification

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my code
• I have commented on my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[linear]

CW-3098 in-app notification if SLA is missed

[sojan-official]

questions

• should we add different SLA missed types for first_response , followup_response and resolution ?
• should we alert all members ? or should me limit to administrators / participants ?

@pranavrajs

---

@vishnu-narayanan

• can we add a spec to ensure that notification is created ?
• can we also test that this won't break email / push notifications since we aren't additing to the notification preferences

[pranavrajs]

should we add different SLA missed types for first_response , followup_response and resolution ?

Yes

should we alert all members ? or should me limit to administrators / participants ?

Participants + Admin

[vishnu-narayanan]

@sojan-official @pranavrajs Done. Could you take another look?

[muhsin-k]

How can we distinguish between different SLA types in the front end if we use the same translation for all types?

[vishnu-narayanan]

It's being used to tag notifications. Adding specific texts would feel repetitive as the notification sub text clearly distinguishes between SLA misses.

[muhsin-k]

Makes sense.

[muhsin-k]

@vishnu-narayanan Please check the comments.

[vishnu-narayanan]

@muhsin-k Done. Could you please review again?

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.