Skip to content

Conversation

@srest2021
Copy link
Member

@srest2021 srest2021 commented Nov 6, 2025

relates to REPLAY-783

Add an AI privacy principles tooltip to the user feedback AI titles, as well as summaries and categories.

Remove the reference to Seer consent in the existing tooltip for spam detection if gen-ai-consent-flow-removal feature flag enabled.

AI titles:

Screenshot 2025-11-07 at 1 29 01 PM

AI summaries & categories:

Screenshot 2025-11-07 at 1 27 43 PM

Spam detection:

hide-ai-features Seer acknowledgement gen-ai-consent-flow-removal Tooltip
true/false false false Screenshot 2025-11-06 at 2 59 58 PM
true true false Screenshot 2025-11-06 at 3 11 46 PM
false true false no tooltip
true true/false true Screenshot 2025-11-06 at 3 12 53 PM
false true/false true no tooltip

Before:
Screenshot 2025-11-07 at 10 22 48 AM

After:
Screenshot 2025-11-07 at 10 22 25 AM

@linear
Copy link

linear bot commented Nov 6, 2025

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 6, 2025
@srest2021 srest2021 marked this pull request as ready for review November 6, 2025 21:27
@srest2021 srest2021 requested a review from a team as a code owner November 6, 2025 21:27
@codecov
Copy link

codecov bot commented Nov 6, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
12354 1 12353 10
View the top 1 failed test(s) by shortest run time
BalanceChangeAction re-enables form after error
Stack Traces | 1.1s run time
Error: expect(element).toHaveValue(10)

Expected the element to have value:
  10
Received:
  1

Ignored nodes: comments, script, style
...
    at toHaveValue (.../gsAdmin/components/changeBalanceAction.spec.tsx:184:45)
    at runWithExpensiveErrorDiagnosticsDisabled (.../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/config.js:47:12)
    at checkCallback (.../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/wait-for.js:124:77)
    at checkRealTimersCallback (.../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/wait-for.js:118:16)
    at Timeout.task [as _onTimeout] (.../sentry/node_modules/.pnpm/jsdom@26.1..../jsdom/browser/Window.js:579:19)
    at listOnTimeout (node:internal/timers:588:17)
    at processTimers (node:internal/timers:523:7)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@srest2021 srest2021 marked this pull request as draft November 6, 2025 21:28
@srest2021 srest2021 marked this pull request as ready for review November 6, 2025 21:43
@srest2021 srest2021 requested a review from malwilley November 6, 2025 21:46
@srest2021 srest2021 changed the title ref(feedback): Update Seer consent flow for user feedback AI title generation ref(feedback): Update Seer consent flow for user feedback AI titles, summaries & categories, and spam detection Nov 6, 2025
@srest2021 srest2021 marked this pull request as draft November 6, 2025 22:54
@michellewzhang
Copy link
Member

michellewzhang commented Nov 7, 2025

ooh okay, so basically: if the user has all the necessary flags turned on, they won't see the tooltip? (for spam detection)

Copy link
Member

@michellewzhang michellewzhang left a comment

Choose a reason for hiding this comment

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

lgtm, do we still plan to add the tooltip to the pill?

@srest2021
Copy link
Member Author

@michellewzhang I think we decided to put it in the text underneath the settings toggle. Adding it to the existing tooltip on the pill would be fine too! I'm not super opinionated on this

@michellewzhang
Copy link
Member

@michellewzhang I think we decided to put it in the text underneath the settings toggle. Adding it to the existing tooltip on the pill would be fine too! I'm not super opinionated on this

that's fine w me!

@srest2021 srest2021 marked this pull request as ready for review November 8, 2025 00:01
@srest2021 srest2021 merged commit dd72927 into master Nov 8, 2025
47 checks passed
@srest2021 srest2021 deleted the srest2021/REPLAY-783-user-feedback branch November 8, 2025 00:21
Jesse-Box pushed a commit that referenced this pull request Nov 12, 2025
…summaries & categories, and spam detection (#102912)

relates to REPLAY-783

Add an AI privacy principles tooltip to the user feedback AI titles, as
well as summaries and categories.

Remove the reference to Seer consent in the existing tooltip for spam
detection if `gen-ai-consent-flow-removal` feature flag enabled.

### AI titles:
<img width="403" height="144" alt="Screenshot 2025-11-07 at 1 29 01 PM"
src="https://github.com/user-attachments/assets/9c4b6106-35b6-4520-8e4a-3c4d898d34dc"
/>

### AI summaries & categories:
<img width="608" height="168" alt="Screenshot 2025-11-07 at 1 27 43 PM"
src="https://github.com/user-attachments/assets/cc0dd9e2-bbf7-4425-ab7b-5b7ff2074109"
/>


### Spam detection:

| `hide-ai-features` | Seer acknowledgement |
`gen-ai-consent-flow-removal` | Tooltip |

|-------------------|------------------------|-------------------------------|---------|
| true/false | false | false | <img width="310" height="152"
alt="Screenshot 2025-11-06 at 2 59 58 PM"
src="https://github.com/user-attachments/assets/2051423e-6e9d-4e49-836b-0a49e90f5ecd"
/> |
| true | true | false | <img width="310" height="152" alt="Screenshot
2025-11-06 at 3 11 46 PM"
src="https://github.com/user-attachments/assets/0ba6563f-3d2f-4a17-adf3-192d6f054f3a"
/> |
| false | true | false | no tooltip |
| true | true/false | true | <img width="310" height="152"
alt="Screenshot 2025-11-06 at 3 12 53 PM"
src="https://github.com/user-attachments/assets/c405427c-c664-401d-a487-24e3a466a0cc"
/> |
| false | true/false | true | no tooltip |

Before:
<img width="768" height="83" alt="Screenshot 2025-11-07 at 10 22 48 AM"
src="https://github.com/user-attachments/assets/c241b042-1158-4fc5-90ef-fda7154401f0"
/>

After:
<img width="778" height="112" alt="Screenshot 2025-11-07 at 10 22 25 AM"
src="https://github.com/user-attachments/assets/f121f68d-ce31-4b5a-a3bc-b2a2c9340753"
/>
andrewshie-sentry pushed a commit that referenced this pull request Nov 13, 2025
…summaries & categories, and spam detection (#102912)

relates to REPLAY-783

Add an AI privacy principles tooltip to the user feedback AI titles, as
well as summaries and categories.

Remove the reference to Seer consent in the existing tooltip for spam
detection if `gen-ai-consent-flow-removal` feature flag enabled.

### AI titles:
<img width="403" height="144" alt="Screenshot 2025-11-07 at 1 29 01 PM"
src="https://github.com/user-attachments/assets/9c4b6106-35b6-4520-8e4a-3c4d898d34dc"
/>

### AI summaries & categories:
<img width="608" height="168" alt="Screenshot 2025-11-07 at 1 27 43 PM"
src="https://github.com/user-attachments/assets/cc0dd9e2-bbf7-4425-ab7b-5b7ff2074109"
/>


### Spam detection:

| `hide-ai-features` | Seer acknowledgement |
`gen-ai-consent-flow-removal` | Tooltip |

|-------------------|------------------------|-------------------------------|---------|
| true/false | false | false | <img width="310" height="152"
alt="Screenshot 2025-11-06 at 2 59 58 PM"
src="https://github.com/user-attachments/assets/2051423e-6e9d-4e49-836b-0a49e90f5ecd"
/> |
| true | true | false | <img width="310" height="152" alt="Screenshot
2025-11-06 at 3 11 46 PM"
src="https://github.com/user-attachments/assets/0ba6563f-3d2f-4a17-adf3-192d6f054f3a"
/> |
| false | true | false | no tooltip |
| true | true/false | true | <img width="310" height="152"
alt="Screenshot 2025-11-06 at 3 12 53 PM"
src="https://github.com/user-attachments/assets/c405427c-c664-401d-a487-24e3a466a0cc"
/> |
| false | true/false | true | no tooltip |

Before:
<img width="768" height="83" alt="Screenshot 2025-11-07 at 10 22 48 AM"
src="https://github.com/user-attachments/assets/c241b042-1158-4fc5-90ef-fda7154401f0"
/>

After:
<img width="778" height="112" alt="Screenshot 2025-11-07 at 10 22 25 AM"
src="https://github.com/user-attachments/assets/f121f68d-ce31-4b5a-a3bc-b2a2c9340753"
/>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants