Commit 7f8e673
ref(feedback): Extracts FeedbackWidgetProvider in a separate file (#4718)
* Update the client implementation to use the new capture feedback js api
* Updates SDK API
* Adds new feedback button in the sample
* Adds changelog
* Removes unused mock
* Update CHANGELOG.md
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com>
* Directly use captureFeedback from sentry/core
* Use import from core
* Fixes imports order lint issue
* Fixes build issue
* Adds captureFeedback tests from sentry-javascript
* Update CHANGELOG.md
* Only deprecate client captureUserFeedback
* Add simple form UI
* Adds basic form functionality
* Update imports
* Update imports
* Remove useState hook to avoid multiple react instances issues
* Move types and styles in different files
* Removes attachment button to be added back separately along with the implementation
* Add basic field validation
* Adds changelog
* Updates changelog
* Updates changelog
* Trim whitespaces from the submitted feedback
* Adds tests
* Renames FeedbackFormScreen to FeedbackForm
* Add beta label
* Extract default text to constants
* Moves constant to a separate file and aligns naming with JS
* Adds input text labels
* Close screen before sending the feedback to minimise wait time
Co-authored-by: LucasZF <lucas-zimerman1@hotmail.com>
* Rename file for consistency
* Flatten configuration hierarchy and clean up
* Align required values with JS
* Use Sentry user email and name when set
* Simplifies email validation
* Show success alert message
* Aligns naming with JS and unmounts the form by default
* Use the minimum config without props in the changelog
* Adds development not for unimplemented function
* Show email and name conditionally
* Adds sentry branding (png logo)
* Adds sentry logo resource
* Add assets in module exports
* Revert "Add assets in module exports"
This reverts commit 5292475.
* Revert "Adds sentry logo resource"
This reverts commit d6e9229.
* Revert "Adds sentry branding (png logo)"
This reverts commit 8c56753.
* Add last event id
* Mock lastEventId
* Adds beta note in the changelog
* Autoinject feedback form
* Updates changelog
* Align colors with JS
* Update CHANGELOG.md
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com>
* Update CHANGELOG.md
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com>
* Update CHANGELOG.md
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com>
* Use regular fonts for both buttons
* Handle keyboard properly
* Adds an option on whether the email should be validated
* Merge properties only once
* Loads current user data on form construction
* Remove unneeded extra padding
* Fix background color issue
* Adds feedback button
* Updates the changelog
* Fixes changelog typo
* Updates styles background color
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com>
* Use defaultProps
* Correct defaultProps
* Adds test to verify when getUser is called
* Use smaller image
Co-authored-by: LucasZF <lucas-zimerman1@hotmail.com>
* Add margin next to the icon
* Adds bottom spacing in the ErrorScreen so that the feedback button does not hide the scrollview buttons
* (2.2) feat: Add Feedback Form UI Branding logo (#4357)
* Adds sentry branding logo as a base64 encoded png
---------
Co-authored-by: LucasZF <lucas-zimerman1@hotmail.com>
* Autoinject feedback form (#4370)
* Align changelog entry
* Update changelog
* Disable bouncing
* Add modal ui appearance
* Update snapshot tests
* Fix bottom margin
* Fix sheet height
* Remove extra modal border
* Do not expose modal styles
* Animate background color
* Avoid keyboard in modal
* Update changelog
* Fix changelog
* Updates comment
* Extract FeedbackButtonProps
* Add public function description to satisfy lint check
* Adds tests
* Fix tests
* Add hardcoded dark and light color themes
* Rename theme options
* Update snapshot tests
* Include in the feedback integration
* Fix circular dependency
* Add theme integration options
* Adds changelog
* Add comment note
* Align with JS api
* Remove unneeded line
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com>
* Place widget button below the feedback widget shadow
* Expose showFeedbackButton/hideFeedbackButton methods
* Add dummy integration for tracking usage
* Adds button border
* Fixes tests
* Add accentBackground and accentForeground colors
* Extract integration getter in a helper function
* Adds dynamic theming support
* Add snapshot tests
* Show screenshot button UI
* Add screenshot button integration
* Add screenshot icon
* Adds Take a screenshot button in FeedbackWidget
* Updates snapshot tests
* Fix circularDepCheck
* Fix circularDepCheck
* Attache captured screenshot
* Hide the take screenshot button when there is a screenshot
* Convert uint8Array to Base64 on the native side
* ref(feedback): Extracts FeedbackWidgetProvider in a separate file
* Updates exposed comments
* Adds snapshot tests
* Disable functionality on the Web
* Add screenshot button in the sample expo app
* Adds system theme tests
* Test dynamically changed theme
* Remove showScreenshotButton and hideScreenshotButton from the exposed api
* Fix function name typo
* Adds enableTakeScreenshot option
* Adds happy flow test
* Make flow tests more granular
* Increate wait time out to fix flakiness on ci
* Reset widget state after each test
* Fix CI flakiness
* Remove flaky test
---------
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com>
Co-authored-by: LucasZF <lucas-zimerman1@hotmail.com>1 parent 4c988a8 commit 7f8e673
File tree
7 files changed
+254
-234
lines changed- packages/core
- src/js
- feedback
- test/feedback
7 files changed
+254
-234
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
12 | | - | |
13 | | - | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | 13 | | |
19 | 14 | | |
20 | 15 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
| 26 | + | |
26 | 27 | | |
27 | 28 | | |
28 | 29 | | |
| |||
335 | 336 | | |
336 | 337 | | |
337 | 338 | | |
338 | | - | |
339 | | - | |
340 | 339 | | |
341 | 340 | | |
342 | 341 | | |
| |||
Lines changed: 26 additions & 223 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
3 | | - | |
4 | | - | |
5 | 2 | | |
6 | | - | |
7 | | - | |
8 | | - | |
9 | | - | |
10 | | - | |
11 | | - | |
12 | | - | |
| 3 | + | |
13 | 4 | | |
14 | | - | |
15 | | - | |
16 | 5 | | |
17 | | - | |
18 | | - | |
19 | | - | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
20 | 9 | | |
21 | 10 | | |
22 | 11 | | |
| |||
65 | 54 | | |
66 | 55 | | |
67 | 56 | | |
68 | | - | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
69 | 64 | | |
70 | 65 | | |
71 | 66 | | |
72 | 67 | | |
73 | 68 | | |
74 | | - | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
75 | 76 | | |
76 | 77 | | |
77 | 78 | | |
78 | 79 | | |
79 | 80 | | |
80 | | - | |
81 | | - | |
82 | | - | |
83 | | - | |
84 | | - | |
85 | | - | |
86 | | - | |
87 | | - | |
88 | | - | |
89 | | - | |
90 | | - | |
91 | | - | |
92 | | - | |
93 | | - | |
94 | | - | |
95 | | - | |
96 | | - | |
97 | | - | |
98 | | - | |
99 | | - | |
100 | | - | |
101 | | - | |
102 | | - | |
103 | | - | |
104 | | - | |
105 | | - | |
106 | | - | |
107 | | - | |
108 | | - | |
109 | | - | |
110 | | - | |
111 | | - | |
112 | | - | |
113 | | - | |
114 | | - | |
115 | | - | |
116 | | - | |
117 | | - | |
118 | | - | |
119 | | - | |
120 | | - | |
121 | | - | |
122 | | - | |
123 | | - | |
124 | | - | |
125 | | - | |
126 | | - | |
127 | | - | |
128 | | - | |
129 | | - | |
130 | | - | |
131 | | - | |
132 | | - | |
133 | | - | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | | - | |
139 | | - | |
140 | | - | |
141 | | - | |
142 | | - | |
143 | | - | |
144 | | - | |
145 | | - | |
146 | | - | |
147 | | - | |
148 | | - | |
149 | | - | |
150 | | - | |
151 | | - | |
152 | | - | |
153 | | - | |
154 | | - | |
155 | | - | |
156 | | - | |
157 | | - | |
158 | | - | |
159 | | - | |
160 | | - | |
161 | | - | |
162 | | - | |
163 | | - | |
164 | | - | |
165 | | - | |
166 | | - | |
167 | | - | |
168 | | - | |
169 | | - | |
170 | | - | |
171 | | - | |
172 | | - | |
173 | | - | |
174 | | - | |
175 | | - | |
176 | | - | |
177 | | - | |
178 | | - | |
179 | | - | |
180 | | - | |
181 | | - | |
182 | | - | |
183 | | - | |
184 | | - | |
185 | | - | |
186 | | - | |
187 | | - | |
188 | | - | |
189 | | - | |
190 | | - | |
191 | | - | |
192 | | - | |
193 | | - | |
194 | | - | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
195 | 85 | | |
196 | | - | |
| 86 | + | |
197 | 87 | | |
198 | | - | |
199 | | - | |
200 | | - | |
201 | | - | |
202 | | - | |
203 | | - | |
204 | | - | |
205 | | - | |
206 | | - | |
207 | | - | |
208 | | - | |
209 | | - | |
210 | | - | |
211 | | - | |
212 | | - | |
213 | | - | |
214 | | - | |
215 | | - | |
216 | | - | |
217 | | - | |
218 | | - | |
219 | | - | |
220 | | - | |
221 | | - | |
222 | | - | |
223 | | - | |
224 | | - | |
225 | | - | |
226 | | - | |
227 | | - | |
228 | | - | |
229 | | - | |
230 | | - | |
231 | | - | |
232 | | - | |
233 | | - | |
234 | | - | |
235 | | - | |
236 | | - | |
237 | | - | |
238 | | - | |
239 | | - | |
240 | | - | |
241 | | - | |
242 | | - | |
| 88 | + | |
| 89 | + | |
243 | 90 | | |
244 | | - | |
245 | | - | |
246 | | - | |
247 | | - | |
248 | | - | |
249 | | - | |
250 | | - | |
251 | | - | |
252 | | - | |
253 | | - | |
254 | | - | |
255 | | - | |
256 | | - | |
257 | | - | |
258 | | - | |
259 | | - | |
260 | | - | |
261 | | - | |
262 | | - | |
263 | | - | |
264 | | - | |
265 | | - | |
266 | | - | |
267 | | - | |
268 | | - | |
269 | | - | |
270 | | - | |
271 | | - | |
272 | | - | |
273 | | - | |
274 | | - | |
275 | | - | |
276 | | - | |
277 | | - | |
278 | | - | |
279 | | - | |
280 | | - | |
281 | | - | |
282 | | - | |
283 | | - | |
284 | | - | |
285 | | - | |
286 | | - | |
287 | | - | |
288 | 91 | | |
289 | 92 | | |
290 | 93 | | |
| |||
326 | 129 | | |
327 | 130 | | |
328 | 131 | | |
329 | | - | |
| 132 | + | |
0 commit comments