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.

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

fix: optimize block template description #4721

Merged
merged 4 commits into from
Jun 21, 2024
Merged

fix: optimize block template description #4721

merged 4 commits into from
Jun 21, 2024

Conversation

katherinehhh
Copy link
Contributor

@katherinehhh katherinehhh commented Jun 20, 2024

Description

Motivation

优化使用区块模板时的显示

没有安装多数据源时: 表名 ; 引用模板: 模板名
安装了多数据源时:数据源 > 表名 ; 引用模板:模板名

Key changes

  • Frontend
  • Backend

Test plan

Suggestions

Underlying risk

Showcase

Copy link

height bot commented Jun 20, 2024

This pull request has been linked to 1 task:

  • T-4620 区块模板的说明需要简化,现在有些信息有点多余,而且格式和以前不一样了 (unlink task)

💡Tip: Add "Close T-4620" to the pull request title or description, to a commit message, or in a comment to mark this task as "Done" when the pull request is merged.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

Copy link

Tests results for "E2E"

8 flaky ⚠️ [chromium] › core/client/src/modules/variable/__e2e__/basic.test.ts:14:7 › variables › linkage rules of table view action
⚠️ [chromium] › plugins/@nocobase/plugin-action-bulk-edit/src/client/__e2e__/schemaInitailizer.test.ts:14:7 › TableActionInitializers & GanttActionInitializers & MapActionInitializers should add bulk edit action › bulk edit in TableActionInitializers
⚠️ [chromium] › plugins/@nocobase/plugin-action-bulk-update/src/client/__e2e__/schemaInitailizer.test.ts:14:7 › TableActionInitializers & GanttActionInitializers & MapActionInitializers › TableActionInitializers should add bulk update
⚠️ [chromium] › plugins/@nocobase/plugin-auth/src/client/__e2e__/auth.test.ts:21:7 › auth › register
⚠️ [chromium] › plugins/@nocobase/plugin-file-manager/src/client/__e2e__/editLocalStorage.test.ts:15:7 › File manager › edit local storage title
⚠️ [chromium] › plugins/@nocobase/plugin-acl/src/client/__e2e__/configure.test.ts:13:5 › allows to configure interface
⚠️ [chromium] › plugins/@nocobase/plugin-action-bulk-update/src/client/__e2e__/schemaSettings.test.ts:14:7 › data will be updated && Assign field values && after successful submission › data will be updated && Assign field values && after successful submission
⚠️ [chromium] › plugins/@nocobase/plugin-action-bulk-edit/src/client/__e2e__/schemaSettings.test.ts:14:7 › bulk edit action setting › data will be updated && edit form configure

987 passed, 30 skipped
✔️✔️✔️

Full HTML report. Merge workflow run.

@katherinehhh katherinehhh merged commit fb1ed31 into main Jun 21, 2024
9 checks passed
@katherinehhh katherinehhh deleted the T-4620 branch June 21, 2024 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant