chatwoot icon indicating copy to clipboard operation
chatwoot copied to clipboard

feat: CSAT Response Visibility

Open JeffOtano opened this issue 1 year ago • 4 comments

Description

This change creates the ability for an administrator to hide the CSAT (Customer Satisfaction Survey) result from agents.

This includes:

  • New UI for the setting under inbox settings
  • New DB Migration to add the field
  • Limit display content attributes based on this setting and whether the viewer is an administrator

Motivation behind this change was derived from one of our users who mentioned that their Agents were complaining about poor CSAT surveys, ideally this is strictly viewed by the management (administrators)

Addresses #9447

Type of change

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

How Has This Been Tested?

This change has gone through manually testing for multiple inbox types that Rinsed uses, specifically:

  • Widget CSAT Response
  • Email CSAT Respone
  • SMS CSAT Response

Checklist:

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

JeffOtano avatar May 22 '24 19:05 JeffOtano

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: app/models/message.rb

Function Unhandled Issue
push_event_data [**NoMethodError: undefined method source_id' for nil:NilClass (NoMethodError)**](https://chatwoot-p3.sentry.io/issues/5283996761/?referrer=github-open-pr-bot) Sidekiq... <br> Event Count:` 1

Did you find this useful? React with a 👍 or 👎

sentry[bot] avatar May 22 '24 19:05 sentry[bot]

@pranavrajs Could I get a quick review on this one? Hoping to cherry pick it off of develop for our local branch!

JeffOtano avatar May 28 '24 19:05 JeffOtano

🐢 Turtley slow progress alert! This pull request has been idle for over 30 days. Can we please speed things up and either merge it or release it back into the wild?

github-actions[bot] avatar Jul 11 '24 03:07 github-actions[bot]

any update on this?

keven-cerrado avatar Aug 19 '24 20:08 keven-cerrado

🐢 Turtley slow progress alert! This pull request has been idle for over 30 days. Can we please speed things up and either merge it or release it back into the wild?

github-actions[bot] avatar Sep 19 '24 03:09 github-actions[bot]