Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add activity message for priority change #6933

Merged
merged 19 commits into from
Apr 20, 2023
Merged

Conversation

scmmishra
Copy link
Member

image

@pr-triage pr-triage bot added the PR: unreviewed This pull request is yet to be reviewed. label Apr 19, 2023
@netlify
Copy link

netlify bot commented Apr 19, 2023

Deploy Preview for chatwoot-storybook ready!

Name Link
🔨 Latest commit 0cd04d8
🔍 Latest deploy log https://app.netlify.com/sites/chatwoot-storybook/deploys/6440ebcbee5fc600087b61cd
😎 Deploy Preview https://deploy-preview-6933--chatwoot-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Comment on lines 72 to 73
typing_status_manager = ::Conversations::FilterService.new(@conversation, current_user, params)
typing_status_manager.toggle_typing_status
Copy link
Member Author

@scmmishra scmmishra Apr 19, 2023

Choose a reason for hiding this comment

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

@sojan-official this change is to manage the rubocop error of too many lines, is it unecessary?

@muhsin-k muhsin-k self-requested a review April 20, 2023 06:23
@muhsin-k
Copy link
Member

muhsin-k commented Apr 20, 2023

@scmmishra

  • I am getting this error on typing.

CleanShot 2023-04-20 at 12 18 44

  • The toggle_status API is not working well.
Screen.Recording.2023-04-20.at.12.20.03.PM.mov
  • The activity message is not coming when I change priority via automation.

@scmmishra
Copy link
Member Author

@muhsin-k noted, fixing that

Copy link
Member

@muhsin-k muhsin-k left a comment

Choose a reason for hiding this comment

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

@pr-triage pr-triage bot removed the PR: unreviewed This pull request is yet to be reviewed. label Apr 20, 2023
@pr-triage pr-triage bot added the PR: partially-approved Not all reviewers have approved the PR label Apr 20, 2023
@scmmishra scmmishra merged commit a34729c into develop Apr 20, 2023
@scmmishra scmmishra deleted the feature/cw-1528 branch April 20, 2023 11:11
@pr-triage pr-triage bot added PR: merged The pull request is merged to another branch and removed PR: partially-approved Not all reviewers have approved the PR labels Apr 20, 2023
@github-actions
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: merged The pull request is merged to another branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants