fix(manager): replace manager contact us with just a sentry feedback flow

[ohnorobo]

Replace the "Contact Us" dialog tree with just the "Feedback" dialog in the manager. That still uses Sentry.captureFeedback.

Currently we give the strong impression that users can get in contact with us directly for support. Since we no longer offer this it's best to remove the meandering dialog flow and just offer a Feedback option.

Should be followed by #2755, but I split that into its own PR to make this more readable.

Feedback flow (tested via npm run action server_manager/www/start):

TODO: also do this for the client once #2739 is merged.

[greptile-apps]

Greptile Summary

This PR simplifies the server manager's feedback flow by removing the multi-step "Contact Us" dialog tree (outline-contact-us-dialog, outline-support-form) and replacing it unconditionally with the existing simpler outline-feedback-dialog that submits via Sentry.captureFeedback. The contactView feature flag and its conditional rendering logic are fully removed.

Key changes:

• outline-contact-us-dialog.ts and outline-support-form.ts are deleted; no remaining references exist in the codebase.
• app-root.ts: the contactView feature flag, both <template is=\"dom-if\"> dialog branches, the nav-label conditional, and the now-unused showContactSuccessToast/showContactErrorToast methods are all removed. A single <outline-feedback-dialog id=\"feedbackDialog\"> replaces both.
• client/web/views/contact_view/index.ts: a TODO comment is added pointing to the follow-up PR (chore(client): use Sentry User Feedback API for client feedback #2739) to apply the same simplification to the client.
• The nav-contact-us localization key is now unused in the manager (the client-side removal is deferred to PR chore(client): use Sentry User Feedback API for client feedback #2739). The key remains in all message files but is harmless dead weight for now.

Confidence Score: 5/5

Safe to merge — a clean, well-scoped removal of a deprecated dialog with no remaining broken references.

All references to the deleted components are gone, the feature flag and its conditional branches are fully removed, the replacement dialog was already in use and proven, and the author tested the flow manually. The only leftover is the now-unused nav-contact-us string in message files, which is harmless and deferred to a follow-up cleanup.

No files require special attention.

Important Files Changed

Filename	Overview
server_manager/www/ui_components/app-root.ts	Removes contactView feature flag, its two conditional dialog templates, and the now-unused success/error toast handlers; replaces them with a single unconditional outline-feedback-dialog. The simplification is correct and complete.
server_manager/www/ui_components/outline-contact-us-dialog.ts	Entire file deleted; no remaining references exist in the codebase. Deletion is clean.
server_manager/www/ui_components/outline-support-form.ts	Entire file deleted; no remaining references exist in the codebase. Deletion is clean.
client/web/views/contact_view/index.ts	Only change is a TODO comment linking to the follow-up PR (#2739) to remove this view from the client side. No functional changes.

Sequence Diagram

sequenceDiagram
    participant User
    participant AppRoot
    participant FeedbackDialog as outline-feedback-dialog
    participant AppTS as app.ts (SubmitFeedback handler)
    participant Sentry

    User->>AppRoot: Tap "Feedback" (nav-feedback)
    AppRoot->>FeedbackDialog: open()
    User->>FeedbackDialog: Fill in form & submit
    FeedbackDialog->>AppRoot: fire('SubmitFeedback', detail)
    AppRoot->>AppTS: event propagates
    AppTS->>Sentry: captureFeedback({ message, email, tags })
    AppTS->>AppRoot: showNotification('notification-feedback-thanks')

Loading

_{Reviews (1): Last reviewed commit: "remove Contact Us dialog flow from the m..." | Re-trigger Greptile}

[fortuna]

Looks good, if we are still showing the toast