Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion frontend/src/i18n/en.js
Original file line number Diff line number Diff line change
Expand Up @@ -550,7 +550,8 @@ const en = {
},
Sentiment: {
averageSentiment: '[Sentiment] Average',
date: '[Sentiment] Date'
date: '[Sentiment] Date',
platform: '[Sentiment] Platform'
}
}
},
Expand Down
12 changes: 7 additions & 5 deletions frontend/src/modules/report/report-charts.js
Original file line number Diff line number Diff line change
Expand Up @@ -95,15 +95,16 @@ export function chartOptions(widget, resultSet) {
) {
const platforms = (
resultSet ? resultSet.tablePivot() : []
).map((p) => p['Activities.platform'])
)
.map((p) => p['Activities.platform'])
.filter((item, i, ar) => ar.indexOf(item) === i)

let mappedColors = platforms.map((p) => {
if (p in platformColors) {
return platformColors[p]
}
return null
return platformColors[p]
})
const defaultColors =
chartTypeOptions.colors || defaultChartOptions.colors

let restColors = defaultColors.filter(
(c) => !mappedColors.includes(c)
)
Expand All @@ -115,6 +116,7 @@ export function chartOptions(widget, resultSet) {
}
return c
})

chartTypeOptions = {
...chartTypeOptions,
colors: [...mappedColors, ...restColors]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,9 @@ export default {
computed: {
model: {
get() {
return this.chartType
return this.modelValue === 'line'
? 'area'
: this.modelValue
},
set(value) {
this.updateChartType(value)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
v-model="value"
clearable
filterable
:disabled="measures.length === 0"
:disabled="computedDimensions.length <= 1"
class="w-full"
@mouseleave="onSelectMouseLeave"
>
Expand Down Expand Up @@ -94,7 +94,8 @@ export default {
'Members.joinedAt',
'Members.organization',
'Tags.name'
]
],
'Sentiment.averageSentiment': ['Sentiment.platform']
}
}
},
Expand All @@ -111,8 +112,26 @@ export default {
},
value: {
get() {
const measure = this.measures[0]

// Select first option by default if measure changes
if (measure) {
const hasOption = this.measureDimensions[
measure.name
]?.includes(this.dimensions?.[0]?.name)

if (
!hasOption &&
this.measureDimensions[measure.name]?.[0]
) {
this.setDimensions([
this.measureDimensions[measure.name][0]
])
}
}

return this.translatedOptions(this.dimensions).map(
(i) => i.label
(i) => i.name
)?.[0]
},
set(value) {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
<template>
<div class="widget-filter-container">
<div
v-if="!!computedFilters.length"
class="widget-filter-container"
>
<div class="flex items-center justify-between">
<div class="font-semibold text-sm">Filters</div>
</div>
Expand Down Expand Up @@ -130,6 +133,28 @@
/>
</el-option-group>
</el-select>
<el-select
v-else-if="filter.select === 'Members.score'"
v-model="filter.value"
class="third-filter"
placeholder="Value"
@change="
(value) =>
handleFilterChange(
'third-option',
value,
index
)
"
>
<el-option
v-for="engagementLevel of computedEngagementLevelTypes"
:key="engagementLevel.label"
:label="engagementLevel.label"
:value="engagementLevel.label"
@mouseleave="onSelectMouseLeave"
/>
</el-select>
<el-input
v-else
v-model="filter.value"
Expand Down Expand Up @@ -180,8 +205,10 @@ import { onSelectMouseLeave } from '@/utils/select'
import { mapGetters, mapActions } from 'vuex'
import { CrowdIntegrations } from '@/integrations/integrations-config'
import { ActivityModel } from '@/modules/activity/activity-model'
import { MemberModel } from '@/modules/member/member-model'

const { fields } = ActivityModel
const { fields: activityFields } = ActivityModel
const { fields: memberFields } = MemberModel

export default {
name: 'FilterComponent',
Expand Down Expand Up @@ -296,28 +323,28 @@ export default {
: this.availableDimensions.filter((d) => {
return this.measureDimensionFilters[
measure.name
][dimension] === undefined
]?.[dimension] === undefined
? false
: this.measureDimensionFilters[measure.name][
dimension
].includes(d.name)
})
},
computedActivityTypes() {
return fields.type
return activityFields.type
.dropdownOptions()
.filter((i) =>
Object.keys(this.activeIntegrationsList).includes(
i.label.key
)
)
},
computedEngagementLevelTypes() {
return memberFields.engagementLevel.dropdownOptions()
},
...mapGetters({
activeIntegrationsList: 'integration/activeList'
}),
fields() {
return fields
}
})
},
async created() {
await this.doFetchIntegrations()
Expand Down Expand Up @@ -372,10 +399,19 @@ export default {
].every((value) => value !== '' && value != null)
})
.map((filter) => {
let values = [filter.value]

// Members engagement level needs to be parsed differently
if (filter.select === 'Members.score') {
values = this.computedEngagementLevelTypes
.filter((t) => t.label === filter.value)?.[0]
.value.map((v) => `${v}`)
}

return {
member: filter.select,
operator: filter.operator,
values: [filter.value]
values
}
})

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
"
clearable
filterable
:disabled="computedTimeDimensions.length <= 1"
class="w-full"
@change="handleTimeChange"
>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,19 +60,18 @@
<div class="w-full mb-6">
<ChartType
v-model="model.settings.chartType"
:chart-type="
chartType === 'line'
? 'area'
: chartType
"
:update-chart-type="updateChartType"
></ChartType>
</div>
<div class="w-full mb-6">
<MeasureSelect
:translated-options="translatedOptions"
:measures="measures"
:available-measures="availableMeasures"
:available-measures="
availableMeasures.filter(
(m) => m.name !== 'Identities.count'
)
"
:set-measures="setMeasures"
/>
</div>
Expand Down Expand Up @@ -110,6 +109,7 @@
<div
v-if="
model.settings.chartType === 'area' ||
model.settings.chartType === 'line' ||
model.settings.chartType === 'bar' ||
model.settings.chartType === 'table'
"
Expand Down Expand Up @@ -321,6 +321,7 @@ export default {

const initialCharType =
this.widget.settings?.chartType || 'line'

return {
mapWidget,
chartOptions,
Expand Down
12 changes: 6 additions & 6 deletions frontend/src/modules/widget/components/widget.vue
Original file line number Diff line number Diff line change
@@ -1,10 +1,5 @@
<template>
<div class="widget">
<div
v-show="loading"
v-loading="loading"
class="app-page-spinner"
></div>
<div
v-if="!number"
class="flex items-center leading-normal justify-between"
Expand Down Expand Up @@ -33,7 +28,12 @@
</router-link>
</div>
<div class="pt-4">
<slot></slot>
<div
v-if="loading"
v-loading="loading"
class="app-page-spinner !relative top-2"
></div>
<slot v-else></slot>
</div>
<el-dropdown
v-if="config.settings"
Expand Down