From dea6ebd31af8e97e8b14160181bd4ea7578c94e8 Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Mon, 30 Sep 2019 12:06:01 +0000 Subject: [PATCH] Add latest changes from gitlab-org/gitlab@master --- .gitlab/ci/review.gitlab-ci.yml | 2 +- README.md | 4 +- .../cycle_analytics/mixins/add_stage_mixin.js | 11 ----- .../cycle_analytics/cycle_analytics_bundle.js | 10 +---- .../ide/stores/modules/commit/actions.js | 16 +++---- .../components/charts/time_series.vue | 15 ++++++- app/helpers/submodule_helper.rb | 2 +- app/mailers/emails/issues.rb | 4 +- app/mailers/emails/members.rb | 16 ++++--- app/mailers/emails/merge_requests.rb | 2 +- app/mailers/emails/notes.rb | 2 +- app/mailers/emails/pages_domains.rb | 8 ++-- app/mailers/emails/projects.rb | 10 ++--- app/mailers/emails/remote_mirrors.rb | 3 +- app/mailers/notify.rb | 14 ------- app/models/project.rb | 1 - app/services/issues/zoom_link_service.rb | 12 +----- app/services/projects/fork_service.rb | 1 + app/views/layouts/nav/_dashboard.html.haml | 4 +- ...30217-refactor-email-notification-code.yml | 5 +++ .../30881-webide-mr-confirmation.yml | 6 +++ .../32133-remove-feature-flag-for-zoom.yml | 5 +++ changelogs/unreleased/fork_gitlab_ci.yml | 5 +++ ...ternationalization-time-series-wrapper.yml | 5 +++ changelogs/unreleased/issue_11241.yml | 5 +++ doc/api/epics.md | 8 ++-- doc/user/project/quick_actions.md | 4 +- locale/gitlab.pot | 8 +++- .../review-qa-all_master_report.json | 42 ------------------- .../master_report.json} | 8 ++++ qa/qa/page/main/menu.rb | 12 ++++-- .../3_create/snippet/create_snippet_spec.rb | 2 +- spec/helpers/application_helper_spec.rb | 2 + spec/helpers/auto_devops_helper_spec.rb | 2 + spec/helpers/avatars_helper_spec.rb | 2 + spec/helpers/award_emoji_helper_spec.rb | 2 + spec/helpers/blame_helper_spec.rb | 2 + spec/helpers/blob_helper_spec.rb | 2 + spec/helpers/boards_helper_spec.rb | 2 + .../helpers/broadcast_messages_helper_spec.rb | 2 + spec/helpers/button_helper_spec.rb | 2 + spec/helpers/calendar_helper_spec.rb | 2 + spec/helpers/ci_status_helper_spec.rb | 2 + spec/helpers/commits_helper_spec.rb | 2 + spec/helpers/components_helper_spec.rb | 2 + spec/helpers/dashboard_helper_spec.rb | 2 + spec/helpers/defer_script_tag_helper_spec.rb | 2 + spec/helpers/emails_helper_spec.rb | 2 + spec/helpers/events_helper_spec.rb | 2 + spec/helpers/explore_helper_spec.rb | 2 + spec/helpers/form_helper_spec.rb | 2 + spec/helpers/git_helper_spec.rb | 2 + spec/helpers/gitlab_routing_helper_spec.rb | 2 + spec/helpers/graph_helper_spec.rb | 2 + spec/helpers/hooks_helper_spec.rb | 2 + spec/helpers/icons_helper_spec.rb | 2 + spec/helpers/import_helper_spec.rb | 2 + .../instance_configuration_helper_spec.rb | 2 + spec/helpers/issues_helper_spec.rb | 2 + spec/helpers/labels_helper_spec.rb | 2 + spec/helpers/markup_helper_spec.rb | 8 ++-- spec/helpers/members_helper_spec.rb | 2 + spec/helpers/milestones_helper_spec.rb | 2 + .../helpers/milestones_routing_helper_spec.rb | 2 + spec/helpers/namespaces_helper_spec.rb | 2 + spec/helpers/nav_helper_spec.rb | 2 + spec/helpers/notes_helper_spec.rb | 2 + spec/helpers/notifications_helper_spec.rb | 2 + spec/helpers/page_layout_helper_spec.rb | 2 + spec/helpers/pagination_helper_spec.rb | 2 + spec/helpers/preferences_helper_spec.rb | 2 + spec/helpers/profiles_helper_spec.rb | 2 + spec/helpers/rss_helper_spec.rb | 2 + spec/helpers/runners_helper_spec.rb | 2 + spec/helpers/search_helper_spec.rb | 2 + spec/helpers/sidekiq_helper_spec.rb | 2 + spec/helpers/snippets_helper_spec.rb | 2 + spec/helpers/storage_helper_spec.rb | 2 + spec/helpers/submodule_helper_spec.rb | 2 + spec/helpers/tab_helper_spec.rb | 2 + spec/helpers/time_helper_spec.rb | 2 + spec/helpers/todos_helper_spec.rb | 2 + spec/helpers/tree_helper_spec.rb | 2 + spec/helpers/user_callouts_helper_spec.rb | 2 + spec/helpers/users_helper_spec.rb | 2 + spec/helpers/version_check_helper_spec.rb | 2 + spec/helpers/visibility_level_helper_spec.rb | 2 + spec/helpers/wiki_helper_spec.rb | 2 + .../ide/stores/modules/commit/actions_spec.js | 14 +++---- .../monitoring/charts/time_series_spec.js | 12 ++++++ spec/lib/gitlab/import_export/all_models.yml | 1 - .../services/issues/zoom_link_service_spec.rb | 28 ------------- spec/services/projects/fork_service_spec.rb | 21 ++++++---- spec/support/helpers/gpg_helpers.rb | 2 + .../zoom_quick_actions_shared_examples.rb | 16 ------- 95 files changed, 265 insertions(+), 199 deletions(-) delete mode 100644 app/assets/javascripts/analytics/cycle_analytics/mixins/add_stage_mixin.js create mode 100644 changelogs/unreleased/30217-refactor-email-notification-code.yml create mode 100644 changelogs/unreleased/30881-webide-mr-confirmation.yml create mode 100644 changelogs/unreleased/32133-remove-feature-flag-for-zoom.yml create mode 100644 changelogs/unreleased/fork_gitlab_ci.yml create mode 100644 changelogs/unreleased/internationalization-time-series-wrapper.yml create mode 100644 changelogs/unreleased/issue_11241.yml delete mode 100644 qa/knapsack/gitlab-ce/review-qa-all_master_report.json rename qa/{qa/ce/knapsack/nightly_master_report.json => knapsack/master_report.json} (84%) diff --git a/.gitlab/ci/review.gitlab-ci.yml b/.gitlab/ci/review.gitlab-ci.yml index 01e4b1afdb0..610b0678db1 100644 --- a/.gitlab/ci/review.gitlab-ci.yml +++ b/.gitlab/ci/review.gitlab-ci.yml @@ -209,7 +209,7 @@ review-qa-all: when: manual parallel: 5 script: - - export KNAPSACK_REPORT_PATH=knapsack/${CI_PROJECT_NAME}/review-qa-all_master_report.json + - export KNAPSACK_REPORT_PATH=knapsack/master_report.json - export KNAPSACK_TEST_FILE_PATTERN=qa/specs/features/**/*_spec.rb - gitlab-qa Test::Instance::Any "${QA_IMAGE}" "${CI_ENVIRONMENT_URL}" -- --format RspecJunitFormatter --out tmp/rspec-${CI_JOB_ID}.xml --format html --out tmp/rspec.htm --color --format documentation diff --git a/README.md b/README.md index 153878c5799..95a2192a375 100644 --- a/README.md +++ b/README.md @@ -2,9 +2,9 @@ ## Canonical source -The canonical source of GitLab where development takes place is [hosted on GitLab.com](https://gitlab.com/gitlab-org/gitlab). +The canonical source of GitLab where all development takes place is [hosted on GitLab.com](https://gitlab.com/gitlab-org/gitlab). -A FOSS source of GitLab is hosted on a [mirrored, read-only project on GitLab.com](https://gitlab.com/gitlab-org/gitlab-foss/). +If you wish to clone a copy of GitLab without proprietary code, you can use the read-only mirror of GitLab located at https://gitlab.com/gitlab-org/gitlab-foss/. Please do not submit any issues and/or merge requests to this project. ## Free trial diff --git a/app/assets/javascripts/analytics/cycle_analytics/mixins/add_stage_mixin.js b/app/assets/javascripts/analytics/cycle_analytics/mixins/add_stage_mixin.js deleted file mode 100644 index 6a40f1cbc5e..00000000000 --- a/app/assets/javascripts/analytics/cycle_analytics/mixins/add_stage_mixin.js +++ /dev/null @@ -1,11 +0,0 @@ -export default { - data() { - return { - isCustomStageForm: false, - }; - }, - methods: { - showAddStageForm: () => {}, - hideAddStageForm: () => {}, - }, -}; diff --git a/app/assets/javascripts/cycle_analytics/cycle_analytics_bundle.js b/app/assets/javascripts/cycle_analytics/cycle_analytics_bundle.js index ac903d60089..cd67ba5fab8 100644 --- a/app/assets/javascripts/cycle_analytics/cycle_analytics_bundle.js +++ b/app/assets/javascripts/cycle_analytics/cycle_analytics_bundle.js @@ -3,7 +3,6 @@ import Vue from 'vue'; import Cookies from 'js-cookie'; import { GlEmptyState } from '@gitlab/ui'; import filterMixins from 'ee_else_ce/analytics/cycle_analytics/mixins/filter_mixins'; -import addStageMixin from 'ee_else_ce/analytics/cycle_analytics/mixins/add_stage_mixin'; import Flash from '../flash'; import { __ } from '~/locale'; import Translate from '../vue_shared/translate'; @@ -44,14 +43,8 @@ export default () => { DateRangeDropdown: () => import('ee_component/analytics/shared/components/date_range_dropdown.vue'), 'stage-nav-item': stageNavItem, - CustomStageForm: () => - import('ee_component/analytics/cycle_analytics/components/custom_stage_form.vue'), - AddStageButton: () => - import('ee_component/analytics/cycle_analytics/components/add_stage_button.vue'), - CustomStageFormContainer: () => - import('ee_component/analytics/cycle_analytics/components/custom_stage_form_container.vue'), }, - mixins: [filterMixins, addStageMixin], + mixins: [filterMixins], data() { return { store: CycleAnalyticsStore, @@ -131,7 +124,6 @@ export default () => { return; } - this.hideAddStageForm(); this.isLoadingStage = true; this.store.setStageEvents([], stage); this.store.setActiveStage(stage); diff --git a/app/assets/javascripts/ide/stores/modules/commit/actions.js b/app/assets/javascripts/ide/stores/modules/commit/actions.js index 23caf2d48ed..f767ca92a56 100644 --- a/app/assets/javascripts/ide/stores/modules/commit/actions.js +++ b/app/assets/javascripts/ide/stores/modules/commit/actions.js @@ -152,6 +152,14 @@ export const commitChanges = ({ commit, state, getters, dispatch, rootState, roo branch: getters.branchName, }) .then(() => { + commit(rootTypes.CLEAR_STAGED_CHANGES, null, { root: true }); + + commit(rootTypes.CLEAR_REPLACED_FILES, null, { root: true }); + + setTimeout(() => { + commit(rootTypes.SET_LAST_COMMIT_MSG, '', { root: true }); + }, 5000); + if (state.shouldCreateMR) { const { currentProject } = rootGetters; const targetBranch = getters.isCreatingNewBranch @@ -164,14 +172,6 @@ export const commitChanges = ({ commit, state, getters, dispatch, rootState, roo { root: true }, ); } - - commit(rootTypes.CLEAR_STAGED_CHANGES, null, { root: true }); - - commit(rootTypes.CLEAR_REPLACED_FILES, null, { root: true }); - - setTimeout(() => { - commit(rootTypes.SET_LAST_COMMIT_MSG, '', { root: true }); - }, 5000); }) .then(() => { if (rootGetters.lastOpenedFile) { diff --git a/app/assets/javascripts/monitoring/components/charts/time_series.vue b/app/assets/javascripts/monitoring/components/charts/time_series.vue index 64d0a713b13..04aced985c8 100644 --- a/app/assets/javascripts/monitoring/components/charts/time_series.vue +++ b/app/assets/javascripts/monitoring/components/charts/time_series.vue @@ -1,5 +1,5 @@