Skip to content

Commit 7a22a83

Browse files
committed
working feedback screenshot package with preact
1 parent d024a76 commit 7a22a83

File tree

5 files changed

+26
-1
lines changed

5 files changed

+26
-1
lines changed

packages/feedback/src/index.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,5 @@ export {
44
Feedback,
55
feedbackIntegration,
66
} from './integration';
7+
8+
console.log('Feedback 2');

packages/feedback/src/integration.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,8 @@ export class Feedback implements Integration {
111111
onFormOpen,
112112
onSubmitError,
113113
onSubmitSuccess,
114+
115+
// getScreenshotIntegration,
114116
}: OptionalFeedbackConfiguration = {}) {
115117
// eslint-disable-next-line deprecation/deprecation
116118
this.name = Feedback.id;
@@ -158,6 +160,8 @@ export class Feedback implements Integration {
158160
onFormOpen,
159161
onSubmitError,
160162
onSubmitSuccess,
163+
164+
// getScreenshotIntegration,
161165
};
162166
}
163167

@@ -182,6 +186,7 @@ export class Feedback implements Integration {
182186
this._createWidget(this.options);
183187
} catch (err) {
184188
DEBUG_BUILD && logger.error(err);
189+
console.log(err);
185190
}
186191
}
187192

@@ -241,6 +246,7 @@ export class Feedback implements Integration {
241246
});
242247
} catch (err) {
243248
DEBUG_BUILD && logger.error(err);
249+
console.log(err);
244250
return null;
245251
}
246252
}
@@ -255,6 +261,7 @@ export class Feedback implements Integration {
255261
return this._createWidget(mergeOptions(this.options, optionOverrides || {}));
256262
} catch (err) {
257263
DEBUG_BUILD && logger.error(err);
264+
console.log(err);
258265
return null;
259266
}
260267
}
@@ -281,6 +288,7 @@ export class Feedback implements Integration {
281288
return true;
282289
}
283290
} catch (err) {
291+
console.log(err);
284292
DEBUG_BUILD && logger.error(err);
285293
}
286294

packages/feedback/src/types/index.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,8 @@ export interface FeedbackGeneralConfiguration {
8080
email: string;
8181
name: string;
8282
};
83+
84+
// getScreenshotIntegration: () => Integration;
8385
}
8486

8587
/**

packages/feedback/src/widget/Form.ts

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import type { FeedbackComponent, FeedbackFormData, FeedbackInternalOptions, FeedbackTextConfiguration } from '../types';
22
import { SubmitButton } from './SubmitButton';
33
import { createElement } from './util/createElement';
4+
import * as ScreenshotIntegration from '@sentry-internal/feedback-screenshot';
45

56
export interface FormComponentProps
67
extends Pick<
@@ -139,7 +140,7 @@ export function Form({
139140
name: 'message',
140141
required: true,
141142
className: 'form__input form__input--textarea',
142-
placeholder: messagePlaceholder,
143+
placeholder: 'message placeholder',
143144
});
144145

145146
const cancelEl = createElement(
@@ -155,6 +156,14 @@ export function Form({
155156
cancelButtonLabel,
156157
);
157158

159+
const screenshot = createElement('div', { className: 'btn-group' });
160+
161+
// @ts-expect-error testing
162+
ScreenshotIntegration.feedbackScreenshotIntegration().renderScreenshotWidget({
163+
el: screenshot,
164+
props: null,
165+
});
166+
158167
const formEl = createElement(
159168
'form',
160169
{
@@ -219,6 +228,8 @@ export function Form({
219228
],
220229
),
221230

231+
screenshot,
232+
222233
createElement(
223234
'div',
224235
{

packages/feedback/src/widget/createWidget.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,7 @@ export function createWidget({
7676
} catch (err) {
7777
// TODO: error handling
7878
logger.error(err);
79+
console.log(err);
7980
}
8081
}
8182

@@ -211,6 +212,7 @@ export function createWidget({
211212
} catch (err) {
212213
// TODO: Error handling?
213214
logger.error(err);
215+
console.log(err);
214216
}
215217
}
216218

0 commit comments

Comments
 (0)