Remove `type` property from injected dependencies
As part of a review for: !42115 (diffs) I noticed that the injected dependencies used both type
and required
which is not correct.
Let's remove any instances of this in the application.
Incorrect
inject: {
projectPath: {
default: '',
},
alertId: {
type: String,
default: '',
},
projectId: {
type: String,
default: '',
},
projectIssuesPath: {
type: String,
default: '',
},
},
Correct
inject: {
projectPath: {
default: '',
},
alertId: {
default: '',
},
projectId: {
default: '',
},
projectIssuesPath: {
default: '',
},
},
-
app/assets/javascripts/admin/cohorts/components/usage_ping_disabled.vue -
app/assets/javascripts/admin/dev_ops_report/components/usage_ping_disabled.vue -
app/assets/javascripts/alert_management/components/alert_details.vue -
app/assets/javascripts/alert_management/components/alert_sidebar.vue -
app/assets/javascripts/analytics/product_analytics/components/activity_chart.vue -
app/assets/javascripts/boards/components/board_column.vue -
app/assets/javascripts/boards/components/board_list_header.vue -
app/assets/javascripts/boards/components/board_new_issue.vue -
app/assets/javascripts/boards/components/issue_card_inner.vue -
app/assets/javascripts/boards/components/project_select.vue -
app/assets/javascripts/clusters/forms/components/integration_form.vue -
ee/app/assets/javascripts/analytics/repository_analytics/components/group_repository_analytics.vue -
ee/app/assets/javascripts/environments/components/environment_alert.vue -
ee/app/assets/javascripts/feature_flags/components/strategy.vue -
ee/app/assets/javascripts/pages/admin/users/pipeline_minutes/reset_button.vue -
ee/app/assets/javascripts/vulnerabilities/components/related_issues.vue
Edited by David O'Regan