fix(client): replace contact us flow with a direct feedback form - #2821
Draft
ohnorobo wants to merge 1 commit into
Draft
fix(client): replace contact us flow with a direct feedback form#2821ohnorobo wants to merge 1 commit into
ohnorobo wants to merge 1 commit into
Conversation
ohnorobo
force-pushed
the
laplante/remove-contact-us-client
branch
2 times, most recently
from
July 28, 2026 12:15
4c404ac to
52d06b9
Compare
ohnorobo
force-pushed
the
laplante/remove-contact-us-client
branch
from
July 28, 2026 12:41
52d06b9 to
3929a7d
Compare
ohnorobo
force-pushed
the
laplante/remove-contact-us-client
branch
from
July 28, 2026 14:01
3929a7d to
7bdecc2
Compare
ohnorobo
changed the base branch from
jules-sentry-feedback-14779291792962734435
to
laplante/feedback-strings
July 28, 2026 14:01
ohnorobo
force-pushed
the
laplante/remove-contact-us-client
branch
from
August 3, 2026 10:49
7bdecc2 to
1659d07
Compare
Removes the multi-step "Contact Us" wizard (open-ticket question and help-page redirects) in the client, mirroring the manager change in #2754. The view now shows the feedback form directly, submitting via the Sentry User Feedback API (errorReporter.sendFeedback) introduced in #2739. The form matches the manager's structure: a category selector, an optional email address, and the feedback text — nothing else. The subject, "where did you get your access key" and outreach-consent fields are gone. Guards SupportForm's validity check against running while the form is disconnected (it is now swapped out for a progress spinner during submission), which otherwise threw uncaught. The "Contact us" -> "Feedback" and email-label string changes live in the stacked strings PR (#2822) this is based on. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
ohnorobo
force-pushed
the
laplante/remove-contact-us-client
branch
from
August 3, 2026 11:04
1659d07 to
7badb1d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Client counterpart to #2754 (which did this for the manager). Replaces the multi-step "Contact Us" wizard with a single, direct feedback form — a category selector, an optional email, and the feedback text. Now it mirrors the "Contact Us" form used in the manager.
Changes
contact_view: dropped the "do you have an open ticket?" question, the issue-type help-page redirects, and the exit step. The category selector and support form now render together; the selected category (defaultgeneral) is passed tosendFeedback. Category list trimmed to four client-relevant options.support_form: reduced to an optional email and the feedback text — removed the subject, access-key-source and outreach-consent fields and the redundant "* = Required field" note. Also guardedcheckFormValidity()against running while the form is disconnected (it is swapped for a progress spinner during submission), which was otherwise throwing uncaught.languageCodeproperty and itslanguage-codebinding inapp-root.js.Stacked on the strings PR #2822, where the "Contact us" → "Feedback" rename and the localized message updates live.
Screenshots