From 326e1de67f5dc13071b808ce9e3a2fb9d800f782 Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Fri, 2 Sep 2022 06:12:27 +0000 Subject: [PATCH] Add latest changes from gitlab-org/gitlab@master --- .../notes/components/comment_form.vue | 6 +----- .../javascripts/notes/components/note_body.vue | 2 +- .../javascripts/notes/components/note_form.vue | 4 ++-- .../notes/components/noteable_discussion.vue | 4 ++-- .../notes/components/noteable_note.vue | 8 ++++---- ...831021358_add_index_on_issue_health_status.rb | 16 ++++++++++++++++ db/schema_migrations/20220831021358 | 1 + db/structure.sql | 2 ++ doc/api/graphql/reference/index.md | 3 +++ locale/gitlab.pot | 15 ++++++++++++--- qa/qa/flow/login.rb | 4 +++- qa/qa/flow/merge_request.rb | 4 +++- qa/qa/flow/pipeline.rb | 4 +++- qa/qa/flow/project.rb | 4 +++- qa/qa/flow/purchase.rb | 2 +- qa/qa/flow/saml.rb | 4 +++- qa/qa/flow/settings.rb | 4 +++- qa/qa/flow/sign_up.rb | 4 +++- qa/qa/flow/user.rb | 4 +++- qa/qa/flow/user_onboarding.rb | 4 +++- .../notes/components/comment_form_spec.js | 6 +++--- spec/frontend/notes/components/note_body_spec.js | 10 +++++----- spec/frontend/notes/components/note_form_spec.js | 12 ++++++------ .../notes/components/noteable_discussion_spec.js | 2 +- .../notes/components/noteable_note_spec.js | 2 +- 25 files changed, 88 insertions(+), 43 deletions(-) create mode 100644 db/post_migrate/20220831021358_add_index_on_issue_health_status.rb create mode 100644 db/schema_migrations/20220831021358 diff --git a/app/assets/javascripts/notes/components/comment_form.vue b/app/assets/javascripts/notes/components/comment_form.vue index 61b7f90e87b..bf35d5c3b25 100644 --- a/app/assets/javascripts/notes/components/comment_form.vue +++ b/app/assets/javascripts/notes/components/comment_form.vue @@ -214,11 +214,7 @@ export default { note: { noteable_type: this.noteableType, noteable_id: this.getNoteableData.id, - // Internal notes were identified as `confidential` - // before we decided to treat them as _internal_ - // so now until API is updated we need to use `confidential` - // in request payload. - confidential: this.noteIsInternal, + internal: this.noteIsInternal, note: this.note, }, merge_request_diff_head_sha: this.getNoteableData.diff_head_sha, diff --git a/app/assets/javascripts/notes/components/note_body.vue b/app/assets/javascripts/notes/components/note_body.vue index 57edcadee6f..82c125b79ce 100644 --- a/app/assets/javascripts/notes/components/note_body.vue +++ b/app/assets/javascripts/notes/components/note_body.vue @@ -71,7 +71,7 @@ export default { return this.note.note; }, saveButtonTitle() { - return this.note.confidential ? __('Save internal note') : __('Save comment'); + return this.note.internal ? __('Save internal note') : __('Save comment'); }, hasSuggestion() { return this.note.suggestions && this.note.suggestions.length; diff --git a/app/assets/javascripts/notes/components/note_form.vue b/app/assets/javascripts/notes/components/note_form.vue index fdb2fbe122a..66a28c0a2c8 100644 --- a/app/assets/javascripts/notes/components/note_form.vue +++ b/app/assets/javascripts/notes/components/note_form.vue @@ -136,7 +136,7 @@ export default { ); }, textareaPlaceholder() { - return this.discussionNote?.confidential + return this.discussionNote?.internal ? this.$options.i18n.bodyPlaceholderInternal : this.$options.i18n.bodyPlaceholder; }, @@ -331,7 +331,7 @@ export default {