Add a optional chaining operator to fix sentry errors
What does this MR do and why?
Add an optional chaining operator to fix sentry errors.
Screenshots or screen recordings
No visual changes.
How to set up and validate locally
Implement the following patch:
diff --git a/app/assets/javascripts/vue_merge_request_widget/components/states/work_in_progress.vue b/app/assets/javascripts/vue_merge_request_widget/components/states/work_in_progress.vue
index 4a23f8847d30..267facb0a505 100644
--- a/app/assets/javascripts/vue_merge_request_widget/components/states/work_in_progress.vue
+++ b/app/assets/javascripts/vue_merge_request_widget/components/states/work_in_progress.vue
@@ -29,7 +29,7 @@ export default {
variables() {
return this.mergeRequestQueryVariables;
},
- update: (data) => data.project?.mergeRequest?.userPermissions || {},
+ update: (data) => data.project.mergeRequest.userPermissions,
},
},
props: {
Execute yarn jest spec/frontend/vue_merge_request_widget/components/states/work_in_progress_spec.js
If it fails it works.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #413627 (closed)