feat(browser)!: Remove captureUserFeedback method

Author: andreiborza

dev-packages/browser-integration-tests/suites/public-api/captureUserFeedback/init.js renamed to dev-packages/browser-integration-tests/suites/public-api/captureFeedback/init.js

@@ -1,4 +1,4 @@
-Sentry.captureUserFeedback({
+Sentry.captureFeedback({
   eventId: 'test_event_id',
   email: 'test_email',
   comments: 'test_comments',

dev-packages/browser-integration-tests/suites/public-api/captureUserFeedback/simple_feedback/subject.js renamed to dev-packages/browser-integration-tests/suites/public-api/captureFeedback/simple_feedback/subject.js

@@ -5,7 +5,7 @@ window.Sentry = Sentry;
 Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
   beforeSend(event) {
-    Sentry.captureUserFeedback({
+    Sentry.captureFeedback({
       event_id: event.event_id,
       name: 'John Doe',
       email: 'john@doe.com',

dev-packages/browser-integration-tests/suites/public-api/captureUserFeedback/simple_feedback/test.ts renamed to dev-packages/browser-integration-tests/suites/public-api/captureFeedback/simple_feedback/test.ts

@@ -5,7 +5,7 @@ window.Sentry = Sentry;
 Sentry.init({
   dsn: 'https://public@dsn.ingest.sentry.io/1337',
   beforeSend(event) {
-    Sentry.captureUserFeedback({
+    Sentry.captureFeedback({
       event_id: event.event_id,
       name: 'John Doe',
       email: 'john@doe.com',

dev-packages/browser-integration-tests/suites/public-api/captureUserFeedback/withCaptureException/init.js renamed to dev-packages/browser-integration-tests/suites/public-api/captureFeedback/withCaptureException/init.js

@@ -128,6 +128,10 @@ Sentry.init({
 - The `urlEncode` method has been removed. There is no replacement.
 - The `getDomElement` method has been removed. There is no replacement.
 
+### `@sentry/browser`
+
+- The `captureUserFeedback` method has been removed. Use `captureFeedback` instead as a drop-in replacement.
+
 ### `@sentry/nestjs`
 
 - Removed `WithSentry` decorator. Use `SentryExceptionCaptured` instead.

dev-packages/browser-integration-tests/suites/public-api/captureUserFeedback/withCaptureException/subject.js renamed to dev-packages/browser-integration-tests/suites/public-api/captureFeedback/withCaptureException/subject.js

@@ -8,14 +8,11 @@ import type {
   ParameterizedString,
   Scope,
   SeverityLevel,
-  UserFeedback,
 } from '@sentry/core';
-import { BaseClient, applySdkMetadata, getSDKSource, logger } from '@sentry/core';
-import { DEBUG_BUILD } from './debug-build';
+import { BaseClient, applySdkMetadata, getSDKSource } from '@sentry/core';
 import { eventFromException, eventFromMessage } from './eventbuilder';
 import { WINDOW } from './helpers';
 import type { BrowserTransportOptions } from './transports/types';
-import { createUserFeedbackEnvelope } from './userfeedback';
 
 /**
  * Configuration options for the Sentry Browser SDK.
@@ -105,28 +102,6 @@ export class BrowserClient extends BaseClient<BrowserClientOptions> {
     return eventFromMessage(this._options.stackParser, message, level, hint, this._options.attachStacktrace);
   }
 
-  /**
-   * Sends user feedback to Sentry.
-   *
-   * @deprecated Use `captureFeedback` instead.
-   */
-  public captureUserFeedback(feedback: UserFeedback): void {
-    if (!this._isEnabled()) {
-      DEBUG_BUILD && logger.warn('SDK not enabled, will not capture user feedback.');
-      return;
-    }
-
-    const envelope = createUserFeedbackEnvelope(feedback, {
-      metadata: this.getSdkMetadata(),
-      dsn: this.getDsn(),
-      tunnel: this.getOptions().tunnel,
-    });
-
-    // sendEnvelope should not throw
-    // eslint-disable-next-line @typescript-eslint/no-floating-promises
-    this.sendEnvelope(envelope);
-  }
-
   /**
    * @inheritDoc
    */

dev-packages/browser-integration-tests/suites/public-api/captureUserFeedback/withCaptureException/test.ts renamed to dev-packages/browser-integration-tests/suites/public-api/captureFeedback/withCaptureException/test.ts

@@ -27,6 +27,7 @@ export {
   addIntegration,
   captureException,
   captureEvent,
+  captureFeedback,
   captureMessage,
   close,
   createTransport,
@@ -92,8 +93,6 @@ export {
   init,
   onLoad,
   showReportDialog,
-  // eslint-disable-next-line deprecation/deprecation
-  captureUserFeedback,
 } from './sdk';
 
 export { breadcrumbsIntegration } from './integrations/breadcrumbs';

dev-packages/browser-integration-tests/suites/public-api/captureUserFeedback/withCaptureMessage/init.js renamed to dev-packages/browser-integration-tests/suites/public-api/captureFeedback/withCaptureMessage/init.js

@@ -2,7 +2,6 @@ import {
   consoleSandbox,
   dedupeIntegration,
   functionToStringIntegration,
-  getClient,
   getCurrentScope,
   getIntegrationsToSetup,
   getReportDialogEndpoint,
@@ -13,7 +12,7 @@ import {
   stackParserFromStackParserOptions,
   supportsFetch,
 } from '@sentry/core';
-import type { Client, DsnLike, Integration, Options, UserFeedback } from '@sentry/core';
+import type { Client, DsnLike, Integration, Options } from '@sentry/core';
 import type { BrowserClientOptions, BrowserOptions } from './client';
 import { BrowserClient } from './client';
 import { DEBUG_BUILD } from './debug-build';
@@ -307,16 +306,3 @@ export function forceLoad(): void {
 export function onLoad(callback: () => void): void {
   callback();
 }
-
-/**
- * Captures user feedback and sends it to Sentry.
- *
- * @deprecated Use `captureFeedback` instead.
- */
-export function captureUserFeedback(feedback: UserFeedback): void {
-  const client = getClient<BrowserClient>();
-  if (client) {
-    // eslint-disable-next-line deprecation/deprecation
-    client.captureUserFeedback(feedback);
-  }
-}