Skip to content

Commit fd7a44b

Browse files
authored
ref(feedback): Rename showScreenshot config to enableScreenshot (#11952)
Before submitting a pull request, please take a look at our [Contributing](https://github.com/getsentry/sentry-javascript/blob/master/CONTRIBUTING.md) guidelines and verify: - [ ] If you've added code that should be tested, please add tests. - [ ] Ensure your code lints and the test suite passes (`yarn lint`) & (`yarn test`).
1 parent c3c2bd3 commit fd7a44b

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

packages/feedback/src/core/integration.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ export const buildFeedbackIntegration = ({
6666
autoInject = true,
6767
showEmail = true,
6868
showName = true,
69-
showScreenshot = true,
69+
enableScreenshot = true,
7070
useSentryUser = {
7171
email: 'email',
7272
name: 'username',
@@ -108,7 +108,7 @@ export const buildFeedbackIntegration = ({
108108
isNameRequired,
109109
showEmail,
110110
showName,
111-
showScreenshot,
111+
enableScreenshot,
112112
useSentryUser,
113113

114114
colorScheme,
@@ -176,7 +176,7 @@ export const buildFeedbackIntegration = ({
176176
};
177177

178178
const _loadAndRenderDialog = async (options: FeedbackInternalOptions): Promise<FeedbackDialog> => {
179-
const screenshotRequired = options.showScreenshot && isScreenshotSupported();
179+
const screenshotRequired = options.enableScreenshot && isScreenshotSupported();
180180
const [modalIntegration, screenshotIntegration] = await Promise.all([
181181
_findIntegration<FeedbackModalIntegration>('FeedbackModal', getModalIntegration, 'feedbackModalIntegration'),
182182
screenshotRequired

packages/types/src/feedback/config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ export interface FeedbackGeneralConfiguration {
4545
* Should the screen shots field be included?
4646
* Screen shots cannot be marked as required
4747
*/
48-
showScreenshot: boolean;
48+
enableScreenshot: boolean;
4949

5050
/**
5151
* Fill in email/name input fields with Sentry user context if it exists.

0 commit comments

Comments
 (0)