Closed
Bug 2036194
Opened 4 months ago
Closed 3 months ago
[Feedback MVP] Send user feedback data via Glean
Categories
(Core :: Machine Learning: Frontend, task)
Core
Machine Learning: Frontend
Tracking
()
VERIFIED
FIXED
153 Branch
People
(Reporter: jlevinsohn, Assigned: echa)
References
Details
(Whiteboard: [aife])
Attachments
(3 files)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
|
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-release+
|
Details | Review |
Send user feedback data from the feedback modal via Glean.
- Feedback is sent when user submits the modal
- Glean payload includes:
- feedback type (thumbs up / thumbs down)
- selected reasons (if applicable)
- freeform feedback text (subject to data review approval)
- Payload size stays within Glean limits
- Data collection category is determined and approved before landing
- If required, data review and access controls are completed
- Instrumentation is gated behind pref
smartwindow.user_feedback_collection - Tests validate telemetry collection
Updated•4 months ago
|
also collect the following metadata:
The model used
Number of turns in chat
Language/locale code
Fx Version
Prompt version
Updated•3 months ago
|
Attachment #9582973 -
Attachment description: Bug 2036194 - Send user feedback data via Glean r?pdahiya → Bug 2036194 - Set up Glean metric for Smart Window user feedback r?pdahiya
Updated•3 months ago
|
status-firefox151:
--- → fix-optional
Pushed by echa@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/9e739d5c7637
https://hg.mozilla.org/integration/autoland/rev/76fe00ec9808
Set up Glean metric for Smart Window user feedback r=pdahiya,chutten,data-stewards,omc-reviewers,aminomancer
Comment 4•3 months ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 3 months ago
status-firefox153:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 153 Branch
Comment 5•3 months ago
|
||
Verified as fixed in our latest Nightly 153.0a1 (2026-05-20)
Comment 6•3 months ago
|
||
firefox-beta Uplift Approval Request
- User impact if declined/Reason for urgency: New metric defined in this patch is needed to collect Smart Window user feedback
- Code covered by automated testing?: yes
- Fix verified in Nightly?: yes
- Needs manual QE testing?: yes
- Steps to reproduce for manual QE testing: Verify new metric smartWindowUserFeedbackData is added to metrics table
- Risk associated with taking this patch: low
- Explanation of risk level: This patch only adds new metric to table, no change in functionality
- String changes made/needed?: no
- Is Android affected?: unknown
Attachment #9588694 -
Flags: approval-mozilla-beta?
Flags: qe-verify+
Original Revision: https://phabricator.services.mozilla.com/D298410
Comment 8•3 months ago
|
||
firefox-release Uplift Approval Request
- User impact if declined/Reason for urgency: New metric defined in this patch is needed to collect Smart Window user feedback
- Code covered by automated testing?: yes
- Fix verified in Nightly?: yes
- Needs manual QE testing?: yes
- Steps to reproduce for manual QE testing: Verify new metric smartWindowUserFeedbackData is added to metrics table
- Risk associated with taking this patch: low
- Explanation of risk level: This patch only adds new metric to table, no change in functionality
- String changes made/needed?: no
- Is Android affected?: unknown
Attachment #9588695 -
Flags: approval-mozilla-release?
Original Revision: https://phabricator.services.mozilla.com/D298410
Updated•3 months ago
|
Attachment #9588694 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•3 months ago
|
Comment 10•3 months ago
|
||
| uplift | ||
Updated•3 months ago
|
QA Whiteboard: [qa-ver-needed-c153/b152] → [qa-ver-needed-c153/b152][uplift]
Comment 11•3 months ago
|
||
Verified as fixed in our latest Beta 152.0b2
Updated•3 months ago
|
Attachment #9588695 -
Flags: approval-mozilla-release? → approval-mozilla-release+
Updated•3 months ago
|
Comment 12•3 months ago
|
||
| uplift | ||
Comment 13•3 months ago
|
||
Verified as fixed in our latest Release 151.0.2
Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-ver-needed-c153/b152][uplift] → [qa-ver-done-c153/b152][uplift]
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•