Skip to content

[FIX] Add check before placing chat on-hold to confirm that contact sent last message#23053

Merged
KevLehman merged 6 commits intodevelopfrom
omnichannel/onhold-check
Sep 3, 2021
Merged

[FIX] Add check before placing chat on-hold to confirm that contact sent last message#23053
KevLehman merged 6 commits intodevelopfrom
omnichannel/onhold-check

Conversation

@murtaza98
Copy link
Contributor

@murtaza98 murtaza98 commented Aug 27, 2021

Proposed changes (including videos or screenshots)

Issue(s)

image
The gif image describes the wrong behavior

The agent can set Hold to a chat even if the customer was the one that sent the last message

Steps to test or reproduce

Further comments

Also converted the endpoint file to typescript

Copy link
Member

@KevLehman KevLehman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The backend part looks good, and I did a small check on the FE, but will prefer @MartinSchoeler or @tiagoevanp 's review to go ahead with this 🎉

@murtaza98 murtaza98 requested a review from tiagoevanp August 31, 2021 06:42
@KevLehman KevLehman merged commit e72aea4 into develop Sep 3, 2021
@KevLehman KevLehman deleted the omnichannel/onhold-check branch September 3, 2021 16:36
@sampaiodiego sampaiodiego mentioned this pull request Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments