diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-02-11 12:08:52 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-02-11 12:08:52 +0000 |
commit | 05b5c609cb8c260b10c2eb1b92b711dc82d32c3f (patch) | |
tree | 05253c66806b17c5b1f9f13addab59524d536fc4 /app | |
parent | 1078b7bf25c2cb6e03c57da9ae25b0512858556f (diff) | |
download | gitlab-ce-05b5c609cb8c260b10c2eb1b92b711dc82d32c3f.tar.gz |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'app')
23 files changed, 371 insertions, 129 deletions
diff --git a/app/assets/javascripts/blob/components/blob_header.vue b/app/assets/javascripts/blob/components/blob_header.vue new file mode 100644 index 00000000000..61a66513838 --- /dev/null +++ b/app/assets/javascripts/blob/components/blob_header.vue @@ -0,0 +1,76 @@ +<script> +import ViewerSwitcher from './blob_header_viewer_switcher.vue'; +import DefaultActions from './blob_header_default_actions.vue'; +import BlobFilepath from './blob_header_filepath.vue'; +import eventHub from '../event_hub'; +import { RICH_BLOB_VIEWER, SIMPLE_BLOB_VIEWER } from './constants'; + +export default { + components: { + ViewerSwitcher, + DefaultActions, + BlobFilepath, + }, + props: { + blob: { + type: Object, + required: true, + }, + hideDefaultActions: { + type: Boolean, + required: false, + default: false, + }, + hideViewerSwitcher: { + type: Boolean, + required: false, + default: false, + }, + }, + data() { + return { + activeViewer: this.blob.richViewer ? RICH_BLOB_VIEWER : SIMPLE_BLOB_VIEWER, + }; + }, + computed: { + showViewerSwitcher() { + return !this.hideViewerSwitcher && Boolean(this.blob.simpleViewer && this.blob.richViewer); + }, + showDefaultActions() { + return !this.hideDefaultActions; + }, + }, + created() { + if (this.showViewerSwitcher) { + eventHub.$on('switch-viewer', this.setActiveViewer); + } + }, + beforeDestroy() { + if (this.showViewerSwitcher) { + eventHub.$off('switch-viewer', this.setActiveViewer); + } + }, + methods: { + setActiveViewer(viewer) { + this.activeViewer = viewer; + }, + }, +}; +</script> +<template> + <div class="js-file-title file-title-flex-parent"> + <blob-filepath :blob="blob"> + <template #filepathPrepend> + <slot name="prepend"></slot> + </template> + </blob-filepath> + + <div class="file-actions d-none d-sm-block"> + <viewer-switcher v-if="showViewerSwitcher" :blob="blob" :active-viewer="activeViewer" /> + + <slot name="actions"></slot> + + <default-actions v-if="showDefaultActions" :blob="blob" :active-viewer="activeViewer" /> + </div> + </div> +</template> diff --git a/app/assets/javascripts/blob/components/blob_header_default_actions.vue b/app/assets/javascripts/blob/components/blob_header_default_actions.vue index b0522c08a65..e526fae0dba 100644 --- a/app/assets/javascripts/blob/components/blob_header_default_actions.vue +++ b/app/assets/javascripts/blob/components/blob_header_default_actions.vue @@ -1,6 +1,13 @@ <script> import { GlButton, GlButtonGroup, GlIcon, GlTooltipDirective } from '@gitlab/ui'; -import { BTN_COPY_CONTENTS_TITLE, BTN_DOWNLOAD_TITLE, BTN_RAW_TITLE } from './constants'; +import { + BTN_COPY_CONTENTS_TITLE, + BTN_DOWNLOAD_TITLE, + BTN_RAW_TITLE, + RICH_BLOB_VIEWER, + SIMPLE_BLOB_VIEWER, +} from './constants'; +import eventHub from '../event_hub'; export default { components: { @@ -16,6 +23,11 @@ export default { type: Object, required: true, }, + activeViewer: { + type: String, + default: SIMPLE_BLOB_VIEWER, + required: false, + }, }, computed: { rawUrl() { @@ -24,10 +36,13 @@ export default { downloadUrl() { return `${this.blob.rawPath}?inline=false`; }, + copyDisabled() { + return this.activeViewer === RICH_BLOB_VIEWER; + }, }, methods: { requestCopyContents() { - this.$emit('copy'); + eventHub.$emit('copy'); }, }, BTN_COPY_CONTENTS_TITLE, @@ -41,6 +56,7 @@ export default { v-gl-tooltip.hover :aria-label="$options.BTN_COPY_CONTENTS_TITLE" :title="$options.BTN_COPY_CONTENTS_TITLE" + :disabled="copyDisabled" @click="requestCopyContents" > <gl-icon name="copy-to-clipboard" :size="14" /> diff --git a/app/assets/javascripts/blob/components/blob_header_viewer_switcher.vue b/app/assets/javascripts/blob/components/blob_header_viewer_switcher.vue index 7acdd574359..13ea87c99b1 100644 --- a/app/assets/javascripts/blob/components/blob_header_viewer_switcher.vue +++ b/app/assets/javascripts/blob/components/blob_header_viewer_switcher.vue @@ -6,6 +6,7 @@ import { SIMPLE_BLOB_VIEWER, SIMPLE_BLOB_VIEWER_TITLE, } from './constants'; +import eventHub from '../event_hub'; export default { components: { @@ -21,25 +22,24 @@ export default { type: Object, required: true, }, - }, - data() { - return { - viewer: this.blob.richViewer ? RICH_BLOB_VIEWER : SIMPLE_BLOB_VIEWER, - }; + activeViewer: { + type: String, + default: SIMPLE_BLOB_VIEWER, + required: false, + }, }, computed: { isSimpleViewer() { - return this.viewer === SIMPLE_BLOB_VIEWER; + return this.activeViewer === SIMPLE_BLOB_VIEWER; }, isRichViewer() { - return this.viewer === RICH_BLOB_VIEWER; + return this.activeViewer === RICH_BLOB_VIEWER; }, }, methods: { switchToViewer(viewer) { - if (viewer !== this.viewer) { - this.viewer = viewer; - this.$emit('switch-viewer', viewer); + if (viewer !== this.activeViewer) { + eventHub.$emit('switch-viewer', viewer); } }, }, diff --git a/app/assets/javascripts/blob/event_hub.js b/app/assets/javascripts/blob/event_hub.js new file mode 100644 index 00000000000..0948c2e5352 --- /dev/null +++ b/app/assets/javascripts/blob/event_hub.js @@ -0,0 +1,3 @@ +import Vue from 'vue'; + +export default new Vue(); diff --git a/app/assets/javascripts/contributors/components/contributors.vue b/app/assets/javascripts/contributors/components/contributors.vue index fb7000ee9ed..caad2a835fa 100644 --- a/app/assets/javascripts/contributors/components/contributors.vue +++ b/app/assets/javascripts/contributors/components/contributors.vue @@ -7,11 +7,13 @@ import { __ } from '~/locale'; import { getSvgIconPathContent } from '~/lib/utils/icon_utils'; import { getDatesInRange } from '~/lib/utils/datetime_utility'; import { xAxisLabelFormatter, dateFormatter } from '../utils'; +import ResizableChartContainer from '~/vue_shared/components/resizable_chart/resizable_chart_container.vue'; export default { components: { GlAreaChart, GlLoadingIcon, + ResizableChartContainer, }, props: { endpoint: { @@ -201,25 +203,35 @@ export default { <div v-else-if="showChart" class="contributors-charts"> <h4>{{ __('Commits to') }} {{ branch }}</h4> <span>{{ __('Excluding merge commits. Limited to 6,000 commits.') }}</span> - <div> + <resizable-chart-container> <gl-area-chart + slot-scope="{ width }" + :width="width" :data="masterChartData" :option="masterChartOptions" :height="masterChartHeight" @created="onMasterChartCreated" /> - </div> + </resizable-chart-container> <div class="row"> - <div v-for="contributor in individualChartsData" :key="contributor.name" class="col-6"> + <div + v-for="(contributor, index) in individualChartsData" + :key="index" + class="col-lg-6 col-12" + > <h4>{{ contributor.name }}</h4> <p>{{ n__('%d commit', '%d commits', contributor.commits) }} ({{ contributor.email }})</p> - <gl-area-chart - :data="contributor.dates" - :option="individualChartOptions" - :height="individualChartHeight" - @created="onIndividualChartCreated" - /> + <resizable-chart-container> + <gl-area-chart + slot-scope="{ width }" + :width="width" + :data="contributor.dates" + :option="individualChartOptions" + :height="individualChartHeight" + @created="onIndividualChartCreated" + /> + </resizable-chart-container> </div> </div> </div> diff --git a/app/assets/javascripts/graphql_shared/fragments/blobviewer.fragment.graphql b/app/assets/javascripts/graphql_shared/fragments/blobviewer.fragment.graphql new file mode 100644 index 00000000000..64c894df115 --- /dev/null +++ b/app/assets/javascripts/graphql_shared/fragments/blobviewer.fragment.graphql @@ -0,0 +1,6 @@ +fragment BlobViewer on SnippetBlobViewer { + collapsed + loadingPartialName + renderError + tooLarge +} diff --git a/app/assets/javascripts/pages/projects/pipelines/charts/index.js b/app/assets/javascripts/pages/projects/pipelines/charts/index.js index 2e7af11c39e..d77b84a3b24 100644 --- a/app/assets/javascripts/pages/projects/pipelines/charts/index.js +++ b/app/assets/javascripts/pages/projects/pipelines/charts/index.js @@ -1,58 +1,3 @@ -import $ from 'jquery'; -import Chart from 'chart.js'; - -import { lineChartOptions } from '~/lib/utils/chart_utils'; - import initProjectPipelinesChartsApp from '~/projects/pipelines/charts/index'; -const SUCCESS_LINE_COLOR = '#1aaa55'; - -const TOTAL_LINE_COLOR = '#707070'; - -const buildChart = (chartScope, shouldAdjustFontSize) => { - const data = { - labels: chartScope.labels, - datasets: [ - { - backgroundColor: SUCCESS_LINE_COLOR, - borderColor: SUCCESS_LINE_COLOR, - pointBackgroundColor: SUCCESS_LINE_COLOR, - pointBorderColor: '#fff', - data: chartScope.successValues, - fill: 'origin', - }, - { - backgroundColor: TOTAL_LINE_COLOR, - borderColor: TOTAL_LINE_COLOR, - pointBackgroundColor: TOTAL_LINE_COLOR, - pointBorderColor: '#EEE', - data: chartScope.totalValues, - fill: '-1', - }, - ], - }; - const ctx = $(`#${chartScope.scope}Chart`) - .get(0) - .getContext('2d'); - - return new Chart(ctx, { - type: 'line', - data, - options: lineChartOptions({ - width: ctx.canvas.width, - numberOfPoints: chartScope.totalValues.length, - shouldAdjustFontSize, - }), - }); -}; - -document.addEventListener('DOMContentLoaded', () => { - const chartsData = JSON.parse(document.getElementById('pipelinesChartsData').innerHTML); - - // Scale fonts if window width lower than 768px (iPad portrait) - const shouldAdjustFontSize = window.innerWidth < 768; - - chartsData.forEach(scope => buildChart(scope, shouldAdjustFontSize)); -}); - document.addEventListener('DOMContentLoaded', initProjectPipelinesChartsApp); diff --git a/app/assets/javascripts/projects/pipelines/charts/components/app.vue b/app/assets/javascripts/projects/pipelines/charts/components/app.vue index 4bd72c405ee..4dc1c512689 100644 --- a/app/assets/javascripts/projects/pipelines/charts/components/app.vue +++ b/app/assets/javascripts/projects/pipelines/charts/components/app.vue @@ -1,17 +1,25 @@ <script> +import dateFormat from 'dateformat'; +import { __, sprintf } from '~/locale'; import { GlColumnChart } from '@gitlab/ui/dist/charts'; +import { getDateInPast } from '~/lib/utils/datetime_utility'; import StatisticsList from './statistics_list.vue'; +import PipelinesAreaChart from './pipelines_area_chart.vue'; import { CHART_CONTAINER_HEIGHT, INNER_CHART_HEIGHT, X_AXIS_LABEL_ROTATION, X_AXIS_TITLE_OFFSET, + CHART_DATE_FORMAT, + ONE_WEEK_AGO_DAYS, + ONE_MONTH_AGO_DAYS, } from '../constants'; export default { components: { StatisticsList, GlColumnChart, + PipelinesAreaChart, }, props: { counts: { @@ -22,6 +30,18 @@ export default { type: Object, required: true, }, + lastWeekChartData: { + type: Object, + required: true, + }, + lastMonthChartData: { + type: Object, + required: true, + }, + lastYearChartData: { + type: Object, + required: true, + }, }, data() { return { @@ -30,10 +50,38 @@ export default { }, }; }, + computed: { + areaCharts() { + const { lastWeek, lastMonth, lastYear } = this.$options.chartTitles; + + return [ + this.buildAreaChartData(lastWeek, this.lastWeekChartData), + this.buildAreaChartData(lastMonth, this.lastMonthChartData), + this.buildAreaChartData(lastYear, this.lastYearChartData), + ]; + }, + }, methods: { mergeLabelsAndValues(labels, values) { return labels.map((label, index) => [label, values[index]]); }, + buildAreaChartData(title, data) { + const { labels, totals, success } = data; + + return { + title, + data: [ + { + name: 'all', + data: this.mergeLabelsAndValues(labels, totals), + }, + { + name: 'success', + data: this.mergeLabelsAndValues(labels, success), + }, + ], + }; + }, }, chartContainerHeight: CHART_CONTAINER_HEIGHT, timesChartOptions: { @@ -45,6 +93,22 @@ export default { nameGap: X_AXIS_TITLE_OFFSET, }, }, + get chartTitles() { + const today = dateFormat(new Date(), CHART_DATE_FORMAT); + const pastDate = timeScale => + dateFormat(getDateInPast(new Date(), timeScale), CHART_DATE_FORMAT); + return { + lastWeek: sprintf(__('Pipelines for last week (%{oneWeekAgo} - %{today})'), { + oneWeekAgo: pastDate(ONE_WEEK_AGO_DAYS), + today, + }), + lastMonth: sprintf(__('Pipelines for last month (%{oneMonthAgo} - %{today})'), { + oneMonthAgo: pastDate(ONE_MONTH_AGO_DAYS), + today, + }), + lastYear: __('Pipelines for last year'), + }; + }, }; </script> <template> @@ -68,5 +132,14 @@ export default { /> </div> </div> + <hr /> + <h4 class="my-4">{{ __('Pipelines charts') }}</h4> + <pipelines-area-chart + v-for="(chart, index) in areaCharts" + :key="index" + :chart-data="chart.data" + > + {{ chart.title }} + </pipelines-area-chart> </div> </template> diff --git a/app/assets/javascripts/projects/pipelines/charts/components/pipelines_area_chart.vue b/app/assets/javascripts/projects/pipelines/charts/components/pipelines_area_chart.vue new file mode 100644 index 00000000000..d701f238a2e --- /dev/null +++ b/app/assets/javascripts/projects/pipelines/charts/components/pipelines_area_chart.vue @@ -0,0 +1,46 @@ +<script> +import { GlAreaChart } from '@gitlab/ui/dist/charts'; +import { s__ } from '~/locale'; +import ResizableChartContainer from '~/vue_shared/components/resizable_chart/resizable_chart_container.vue'; +import { CHART_CONTAINER_HEIGHT } from '../constants'; + +export default { + components: { + GlAreaChart, + ResizableChartContainer, + }, + props: { + chartData: { + type: Array, + required: true, + }, + }, + areaChartOptions: { + xAxis: { + name: s__('Pipeline|Date'), + type: 'category', + }, + yAxis: { + name: s__('Pipeline|Pipelines'), + }, + }, + chartContainerHeight: CHART_CONTAINER_HEIGHT, +}; +</script> +<template> + <div class="prepend-top-default"> + <p> + <slot></slot> + </p> + <resizable-chart-container> + <gl-area-chart + slot-scope="{ width }" + :width="width" + :height="$options.chartContainerHeight" + :data="chartData" + :include-legend-avg-max="false" + :option="$options.areaChartOptions" + /> + </resizable-chart-container> + </div> +</template> diff --git a/app/assets/javascripts/projects/pipelines/charts/constants.js b/app/assets/javascripts/projects/pipelines/charts/constants.js index eeb29370e51..5dbe3c01100 100644 --- a/app/assets/javascripts/projects/pipelines/charts/constants.js +++ b/app/assets/javascripts/projects/pipelines/charts/constants.js @@ -5,3 +5,9 @@ export const INNER_CHART_HEIGHT = 200; export const X_AXIS_LABEL_ROTATION = 45; export const X_AXIS_TITLE_OFFSET = 60; + +export const ONE_WEEK_AGO_DAYS = 7; + +export const ONE_MONTH_AGO_DAYS = 31; + +export const CHART_DATE_FORMAT = 'dd mmm'; diff --git a/app/assets/javascripts/projects/pipelines/charts/index.js b/app/assets/javascripts/projects/pipelines/charts/index.js index b0f5f549980..4ae2b729200 100644 --- a/app/assets/javascripts/projects/pipelines/charts/index.js +++ b/app/assets/javascripts/projects/pipelines/charts/index.js @@ -10,8 +10,23 @@ export default () => { successRatio, timesChartLabels, timesChartValues, + lastWeekChartLabels, + lastWeekChartTotals, + lastWeekChartSuccess, + lastMonthChartLabels, + lastMonthChartTotals, + lastMonthChartSuccess, + lastYearChartLabels, + lastYearChartTotals, + lastYearChartSuccess, } = el.dataset; + const parseAreaChartData = (labels, totals, success) => ({ + labels: JSON.parse(labels), + totals: JSON.parse(totals), + success: JSON.parse(success), + }); + return new Vue({ el, name: 'ProjectPipelinesChartsApp', @@ -31,6 +46,21 @@ export default () => { labels: JSON.parse(timesChartLabels), values: JSON.parse(timesChartValues), }, + lastWeekChartData: parseAreaChartData( + lastWeekChartLabels, + lastWeekChartTotals, + lastWeekChartSuccess, + ), + lastMonthChartData: parseAreaChartData( + lastMonthChartLabels, + lastMonthChartTotals, + lastMonthChartSuccess, + ), + lastYearChartData: parseAreaChartData( + lastYearChartLabels, + lastYearChartTotals, + lastYearChartSuccess, + ), }, }), }); diff --git a/app/assets/javascripts/repository/components/last_commit.vue b/app/assets/javascripts/repository/components/last_commit.vue index c0c599f4b3c..968bd9af84f 100644 --- a/app/assets/javascripts/repository/components/last_commit.vue +++ b/app/assets/javascripts/repository/components/last_commit.vue @@ -108,7 +108,12 @@ export default { class="avatar-cell" /> <span v-else class="avatar-cell user-avatar-link"> - <img :src="$options.defaultAvatarUrl" width="40" height="40" class="avatar s40" /> + <img + :src="commit.authorGravatar || $options.defaultAvatarUrl" + width="40" + height="40" + class="avatar s40" + /> </span> <div class="commit-detail flex-list"> <div class="commit-content qa-commit-content"> diff --git a/app/assets/javascripts/repository/components/table/row.vue b/app/assets/javascripts/repository/components/table/row.vue index f97c8ae1f74..8703796b116 100644 --- a/app/assets/javascripts/repository/components/table/row.vue +++ b/app/assets/javascripts/repository/components/table/row.vue @@ -1,4 +1,5 @@ <script> +import { escapeRegExp } from 'lodash'; import { GlBadge, GlLink, GlSkeletonLoading, GlTooltipDirective, GlLoadingIcon } from '@gitlab/ui'; import { visitUrl } from '~/lib/utils/url_utility'; import TimeagoTooltip from '~/vue_shared/components/time_ago_tooltip.vue'; @@ -105,7 +106,7 @@ export default { return this.isFolder ? 'router-link' : 'a'; }, fullPath() { - return this.path.replace(new RegExp(`^${this.currentPath}/`), ''); + return this.path.replace(new RegExp(`^${escapeRegExp(this.currentPath)}/`), ''); }, shortSha() { return this.sha.slice(0, 8); diff --git a/app/assets/javascripts/repository/graphql.js b/app/assets/javascripts/repository/graphql.js index 6936c08d852..265df20636b 100644 --- a/app/assets/javascripts/repository/graphql.js +++ b/app/assets/javascripts/repository/graphql.js @@ -48,7 +48,7 @@ const defaultClient = createDefaultClient( case 'TreeEntry': case 'Submodule': case 'Blob': - return `${obj.flatPath}-${obj.id}`; + return `${escape(obj.flatPath)}-${obj.id}`; default: // If the type doesn't match any of the above we fallback // to using the default Apollo ID diff --git a/app/assets/javascripts/repository/queries/pathLastCommit.query.graphql b/app/assets/javascripts/repository/queries/pathLastCommit.query.graphql index c812614e94d..a22cadf0e8d 100644 --- a/app/assets/javascripts/repository/queries/pathLastCommit.query.graphql +++ b/app/assets/javascripts/repository/queries/pathLastCommit.query.graphql @@ -10,6 +10,7 @@ query pathLastCommit($projectPath: ID!, $path: String, $ref: String!) { webUrl authoredDate authorName + authorGravatar author { name avatarUrl diff --git a/app/assets/javascripts/snippets/components/snippet_blob_view.vue b/app/assets/javascripts/snippets/components/snippet_blob_view.vue index b91e08a4251..49e0ef35cb8 100644 --- a/app/assets/javascripts/snippets/components/snippet_blob_view.vue +++ b/app/assets/javascripts/snippets/components/snippet_blob_view.vue @@ -1,10 +1,26 @@ <script> import BlobEmbeddable from '~/blob/components/blob_embeddable.vue'; import { SNIPPET_VISIBILITY_PUBLIC } from '../constants'; +import BlobHeader from '~/blob/components/blob_header.vue'; +import GetSnippetBlobQuery from '../queries/snippet.blob.query.graphql'; +import { GlLoadingIcon } from '@gitlab/ui'; export default { components: { BlobEmbeddable, + BlobHeader, + GlLoadingIcon, + }, + apollo: { + blob: { + query: GetSnippetBlobQuery, + variables() { + return { + ids: this.snippet.id, + }; + }, + update: data => data.snippets.edges[0].node.blob, + }, }, props: { snippet: { @@ -12,15 +28,32 @@ export default { required: true, }, }, + data() { + return { + blob: {}, + }; + }, computed: { embeddable() { return this.snippet.visibilityLevel === SNIPPET_VISIBILITY_PUBLIC; }, + isBlobLoading() { + return this.$apollo.queries.blob.loading; + }, }, }; </script> <template> <div> <blob-embeddable v-if="embeddable" class="mb-3" :url="snippet.webUrl" /> + <gl-loading-icon + v-if="isBlobLoading" + :label="__('Loading blob')" + :size="2" + class="prepend-top-20 append-bottom-20" + /> + <article v-else class="file-holder snippet-file-content"> + <blob-header :blob="blob" /> + </article> </div> </template> diff --git a/app/assets/javascripts/snippets/queries/snippet.blob.query.graphql b/app/assets/javascripts/snippets/queries/snippet.blob.query.graphql new file mode 100644 index 00000000000..785c88c185a --- /dev/null +++ b/app/assets/javascripts/snippets/queries/snippet.blob.query.graphql @@ -0,0 +1,24 @@ +#import '~/graphql_shared/fragments/blobviewer.fragment.graphql' + +query SnippetBlobFull($ids: [ID!]) { + snippets(ids: $ids) { + edges { + node { + id + blob { + binary + name + path + rawPath + size + simpleViewer { + ...BlobViewer + } + richViewer { + ...BlobViewer + } + } + } + } + } +} diff --git a/app/controllers/projects/registry/tags_controller.rb b/app/controllers/projects/registry/tags_controller.rb index e572c56adf5..c42e3f6bdba 100644 --- a/app/controllers/projects/registry/tags_controller.rb +++ b/app/controllers/projects/registry/tags_controller.rb @@ -31,12 +31,7 @@ module Projects end def bulk_destroy - unless params[:ids].present? - head :bad_request - return - end - - tag_names = params[:ids] || [] + tag_names = params.require(:ids) || [] if tag_names.size > LIMIT head :bad_request return diff --git a/app/controllers/registrations_controller.rb b/app/controllers/registrations_controller.rb index ed5e39478f1..c54687c432c 100644 --- a/app/controllers/registrations_controller.rb +++ b/app/controllers/registrations_controller.rb @@ -117,8 +117,10 @@ class RegistrationsController < Devise::RegistrationsController end def after_inactive_sign_up_path_for(resource) + # With the current `allow_unconfirmed_access_for` Devise setting in config/initializers/8_devise.rb, + # this method is never called. Leaving this here in case that value is set to 0. Gitlab::AppLogger.info(user_created_message) - dashboard_projects_path + users_almost_there_path end private diff --git a/app/graphql/types/commit_type.rb b/app/graphql/types/commit_type.rb index 392e3ae41c5..eb25e3651a8 100644 --- a/app/graphql/types/commit_type.rb +++ b/app/graphql/types/commit_type.rb @@ -26,6 +26,11 @@ module Types description: 'Rendered HTML of the commit signature' field :author_name, type: GraphQL::STRING_TYPE, null: true, description: 'Commit authors name' + field :author_gravatar, type: GraphQL::STRING_TYPE, null: true, + description: 'Commit authors gravatar', + resolve: -> (commit, args, context) do + GravatarService.new.execute(commit.author_email, 40) + end # models/commit lazy loads the author by email field :author, type: Types::UserType, null: true, diff --git a/app/models/ci/pipeline.rb b/app/models/ci/pipeline.rb index 94b99835a53..bb59fc937b1 100644 --- a/app/models/ci/pipeline.rb +++ b/app/models/ci/pipeline.rb @@ -216,7 +216,7 @@ module Ci end end - after_transition created: :pending do |pipeline| + after_transition created: any - [:failed] do |pipeline| next unless pipeline.bridge_triggered? next if pipeline.bridge_waiting? diff --git a/app/views/projects/pipelines/charts.html.haml b/app/views/projects/pipelines/charts.html.haml index 9542f8c9766..7496ca97d56 100644 --- a/app/views/projects/pipelines/charts.html.haml +++ b/app/views/projects/pipelines/charts.html.haml @@ -1,7 +1,7 @@ - page_title _('CI / CD Charts') -#js-project-pipelines-charts-app{ data: { counts: @counts, success_ratio: success_ratio(@counts), times_chart: { labels: @charts[:pipeline_times].labels, values: @charts[:pipeline_times].pipeline_times } } } - -#charts.ci-charts - %hr - = render 'projects/pipelines/charts/pipelines' +#js-project-pipelines-charts-app{ data: { counts: @counts, success_ratio: success_ratio(@counts), + times_chart: { labels: @charts[:pipeline_times].labels, values: @charts[:pipeline_times].pipeline_times }, + last_week_chart: { labels: @charts[:week].labels, totals: @charts[:week].total, success: @charts[:week].success }, + last_month_chart: { labels: @charts[:month].labels, totals: @charts[:month].total, success: @charts[:month].success }, + last_year_chart: { labels: @charts[:year].labels, totals: @charts[:year].total, success: @charts[:year].success } } } diff --git a/app/views/projects/pipelines/charts/_pipelines.haml b/app/views/projects/pipelines/charts/_pipelines.haml deleted file mode 100644 index afff9e82e45..00000000000 --- a/app/views/projects/pipelines/charts/_pipelines.haml +++ /dev/null @@ -1,37 +0,0 @@ -%h4.mt-4.mb-4= _("Pipelines charts") -%p - - %span.legend-success - = icon("circle") - = s_("Pipeline|success") - - %span.legend-all - = icon("circle") - = s_("Pipeline|all") - -.prepend-top-default - %p.light - = _("Pipelines for last week") - (#{date_from_to(Date.today - 7.days, Date.today)}) - %div - %canvas#weekChart{ height: 200 } - -.prepend-top-default - %p.light - = _("Pipelines for last month") - (#{date_from_to(Date.today - 30.days, Date.today)}) - %div - %canvas#monthChart{ height: 200 } - -.prepend-top-default - %p.light - = _("Pipelines for last year") - %div - %canvas#yearChart.padded{ height: 250 } - --# haml-lint:disable InlineJavaScript -%script#pipelinesChartsData{ type: "application/json" } - - chartData = [] - - [:week, :month, :year].each do |scope| - - chartData.push({ 'scope' => scope, 'labels' => @charts[scope].labels, 'totalValues' => @charts[scope].total, 'successValues' => @charts[scope].success }) - = chartData.to_json.html_safe |