Skip to content

Commit a4d23cd

Browse files
committed
remove class-based integration export
1 parent 57fff8d commit a4d23cd

File tree

4 files changed

+6
-31
lines changed

4 files changed

+6
-31
lines changed

packages/feedback2/src/core/integration.ts

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
import { convertIntegrationFnToClass, defineIntegration, getClient } from '@sentry/core';
2-
import type { Integration, IntegrationClass, IntegrationFn, IntegrationFnResult } from '@sentry/types';
1+
import { defineIntegration, getClient } from '@sentry/core';
2+
import type { IntegrationFn, IntegrationFnResult } from '@sentry/types';
33
import { isBrowser, logger } from '@sentry/utils';
44
import {
55
ACTOR_LABEL,
@@ -275,11 +275,3 @@ export const _feedback2Integration = (({
275275
}) satisfies IntegrationFn;
276276

277277
export const feedback2Integration = defineIntegration(_feedback2Integration);
278-
279-
/**
280-
* @deprecated Use `feedback2ScreenshotIntegration()` instead
281-
*/
282-
// eslint-disable-next-line deprecation/deprecation
283-
export const Feedback2 = convertIntegrationFnToClass('Feedback2', feedback2Integration) as IntegrationClass<
284-
Integration & PublicFeedback2Integration
285-
>;

packages/feedback2/src/index.ts

Lines changed: 3 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,7 @@
11
export { sendFeedback } from './core/sendFeedback';
2-
export {
3-
// eslint-disable-next-line deprecation/deprecation
4-
Feedback2,
5-
feedback2Integration,
6-
} from './core/integration';
7-
8-
export {
9-
// eslint-disable-next-line deprecation/deprecation
10-
feedback2ModalIntegration,
11-
} from './modal/integration';
12-
13-
export {
14-
// eslint-disable-next-line deprecation/deprecation
15-
feedback2ScreenshotIntegration,
16-
} from './screenshot/integration';
2+
export { feedback2Integration } from './core/integration';
3+
export { feedback2ModalIntegration } from './modal/integration';
4+
export { feedback2ScreenshotIntegration } from './screenshot/integration';
175

186
export type { OptionalFeedbackConfiguration } from './types';
19-
207
export type { IFeedback2Integration } from './core/integration';

packages/feedback2/src/screenshot/components/useTakeScreenshot.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ export const useTakeScreenshot = ({ onBeforeScreenshot, onScreenshot, onAfterScr
3939
};
4040

4141
takeScreenshot().catch(_error => {
42+
console.log('some takeScreenshotError', error);
4243
// TODO
4344
});
4445
}, []);

packages/feedback2/src/types/index.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -84,11 +84,6 @@ export interface Dialog {
8484
}
8585

8686
export interface ScreenshotInput {
87-
/**
88-
* The style element for this component
89-
*/
90-
// style: HTMLStyleElement;
91-
9287
/**
9388
* The preact component
9489
*/

0 commit comments

Comments
 (0)