Skip to content

Add a optional chaining operator to fix sentry errors

Savas Vedova requested to merge 413627-type-check into master

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.

Related to #413627 (closed)

Merge request reports