feat: CSAT Response Visibility
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
🔍 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 👎
@pranavrajs Could I get a quick review on this one? Hoping to cherry pick it off of develop for our local branch!
🐢 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?
any update on this?
🐢 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?