Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(pivot-data-table): create pivot form (transform data-table) #5215

Merged
merged 14 commits into from
Dec 13, 2024
Merged

feat(pivot-data-table): create pivot form (transform data-table) #5215

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
f3a7954
fix(pivot-data-table): edit pivot option type & create pivot default …
piggggggggy Dec 12, 2024
1384440
feat(data-table): create data table transform form wrapper
piggggggggy Dec 12, 2024
4383bd8
feat(pivot): create pivot data-table form
piggggggggy Dec 12, 2024
d1a6a99
fix(transform-contents): apply pivot form
piggggggggy Dec 12, 2024
02de55b
chore: remove console.debug
piggggggggy Dec 12, 2024
564b25e
fix(form-wrapper): edit wrapper component structure
piggggggggy Dec 12, 2024
c4105ac
fix(pivot-data-table): create other pivot fields (operator, order-by,…
piggggggggy Dec 12, 2024
9dad6ed
chore: edit typo
piggggggggy Dec 12, 2024
3ec1a44
chore: add annotation
piggggggggy Dec 12, 2024
812d132
chore: edit missing code
piggggggggy Dec 12, 2024
eb2f144
feat: add icons (ic_sort-ascending, ic_sort-descending)
piggggggggy Dec 12, 2024
e3968af
chore: fix style
piggggggggy Dec 12, 2024
df8db1b
chore: apply review
piggggggggy Dec 13, 2024
973f997
chore: fix typo
piggggggggy Dec 13, 2024
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
31 changes: 27 additions & 4 deletions apps/web/src/common/modules/widgets/_components/WidgetFormDataTableCardTransformContents.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,10 @@ import WidgetFormDataTableCardHeaderTitle
from '@/common/modules/widgets/_components/WidgetFormDataTableCardHeaderTitle.vue';
import WidgetFormDataTableCardTransformForm
from '@/common/modules/widgets/_components/WidgetFormDataTableCardTransformForm.vue';
import WidgetFormDataTableCardTransformPivotForm
from '@/common/modules/widgets/_components/WidgetFormDataTableCardTransformPivotForm.vue';
import {
DATA_TABLE_TYPE, DATA_TABLE_FIELD_TYPE,
DATA_TABLE_TYPE, DATA_TABLE_FIELD_TYPE, type DATA_TABLE_OPERATOR, DEFAULT_TRANSFORM_DATA_TABLE_VALUE_MAP,
} from '@/common/modules/widgets/_constants/data-table-constant';
import { useWidgetGenerateStore } from '@/common/modules/widgets/_store/widget-generate-store';
import type {
Expand All @@ -29,7 +31,7 @@ import type {
import type {
DataTableOperator, JoinType, ConcatOptions, JoinOptions, QueryOptions, EvalOptions,
DataTableTransformOptions,
EvaluateExpression, AddLabelsOptions, AdditionalLabels,
EvaluateExpression, AddLabelsOptions, AdditionalLabels, PivotOptions,
} from '@/common/modules/widgets/types/widget-model';
Expand Down Expand Up @@ -127,6 +129,10 @@ const evalState = reactive({
}] as EvalExpressions[],
});
const valueState = reactive({
pivot: DEFAULT_TRANSFORM_DATA_TABLE_VALUE_MAP.PIVOT as Omit<PivotOptions, 'data_table_id'>,
});
const addLabelsState = reactive({
labels: [{
name: '', value: '',
Expand All @@ -148,6 +154,17 @@ const originState = reactive({
const _labels = (props.item.options as DataTableTransformOptions).ADD_LABELS?.labels ?? {};
return Object.entries(_labels).map(([name, value]) => ({ name, value }));
}),
pivot: computed<Omit<PivotOptions, 'data_table_id'>|undefined>(() => {
const _pivot = (props.item.options as DataTableTransformOptions).PIVOT as PivotOptions|undefined;
if (!_pivot) return undefined;
return {
fields: _pivot.fields,
select: _pivot.select,
limit: _pivot.limit,
functions: _pivot.functions,
order_by: _pivot.order_by,
};
}),
});
const setFailStatus = (status: boolean) => {
Expand Down Expand Up @@ -317,6 +334,7 @@ const setInitialDataTableForm = () => {
key: getRandomId(), name: '', fieldType: DATA_TABLE_FIELD_TYPE.DATA, expression: '', isCollapsed: false,
}];
addLabelsState.labels = originState.labels.length ? cloneDeep(originState.labels) : [{ name: '', value: '' }];
valueState.pivot = originState.pivot ? cloneDeep(originState.pivot) : DEFAULT_TRANSFORM_DATA_TABLE_VALUE_MAP.PIVOT;
};
onMounted(() => {
Expand Down Expand Up @@ -351,8 +369,13 @@ defineExpose({
:is-legacy-data-table="state.isLegacyDataTable"
/>
</div>

<widget-form-data-table-card-transform-form :data-table-id="state.dataTableId"
<widget-form-data-table-card-transform-pivot-form v-if="state.operator === DATA_TABLE_OPERATOR.PIVOT"
:data-table-id="state.dataTableId"
:data-table-info.sync="state.dataTableInfo"
:form-data.sync="valueState.pivot"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question: what is this formData prop means?

/>
<widget-form-data-table-card-transform-form v-else
:data-table-id="state.dataTableId"
:operator="state.operator"
:data-table-info.sync="state.dataTableInfo"
:join-type.sync="joinState.joinType"
Expand Down
Loading