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

[Feature][admin] Task submit approval #4070

Merged
merged 28 commits into from
Jan 10, 2025

Conversation

MactavishCui
Copy link
Contributor

@MactavishCui MactavishCui commented Dec 19, 2024

Purpose of the pull request

#4061

Brief change log

Verify this pull request

In configuration center, here is an option to enable task submit approval, when it is enabled:
Task cannot be submitted if there's no approved approval for current task version
069f11dbe0ab529135d338b6b44e314

User can submit an approval by this button, before an approval is created, task will be published first
image

A reviewer is needed, also, user can leave some comments like 'Please take a look'
4629a49e83fcab69f54b909f9af58f9

Approval can be managed in auth center, an approval can be submitted, withdrawn, canceled, approved or rejected.
1850684ee75a8d3174a404138761ef5
5fce8a16f58c5b609138b34035b44ef

The task info can also be checked
efe2638987e327d9cb6b1fc65362cdb

After the approval of current task version is approved, task can be submitted
624375ce4bbba1156a2467179c5496e

When task submit approval is disabled,
9aaccdc3f29910d2bed2996257fd174
Approval create and submit button will not be shown
image
Approval management page is not available neither
c4a10a954a785979949f2e5116c2320
Task can be submitted directly
3466efc79555c0b7a2c069f9fc01f5e

todo:
1.handle reload
2.task info query
3.properties roll back
4.code optimization
5.h2 and pg sql
6.function test
todo:
1.task info query
2.properties roll back
3.code optimization
4.h2 and pg sql
5.function test
todo:
1.properties roll back
2.layout
3.code optimization
4.h2 and pg sql
5.function test
todo:
1.layout
2.code optimization
3.h2 and pg sql
4.function test
todo:
1.code optimization
2.h2 and pg sql
3.function test
todo:
1.code optimization
2.h2 and pg sql
3.function test
todo:
1.code optimization
2.h2 and pg sql
3.function test
@Zzm0809
Copy link
Contributor

Zzm0809 commented Dec 25, 2024

The menu list data needs to be added to the upgrade file of the corresponding db type

@Zzm0809
Copy link
Contributor

Zzm0809 commented Dec 25, 2024

如果平台内只有一个用户时,开启强制交叉审核时,在审批人列表中无法选择审批人

@MactavishCui
Copy link
Contributor Author

@Zzm0809 感谢指正,我将于本周内向此分支提交新的commit来修复上述问题

@MactavishCui MactavishCui marked this pull request as ready for review January 8, 2025 06:23
Copy link
Contributor

@Zzm0809 Zzm0809 left a comment

Choose a reason for hiding this comment

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

LGTM
@gaoyan1998 please review again

@Zzm0809 Zzm0809 merged commit bbc4db4 into DataLinkDC:dev Jan 10, 2025
22 checks passed
@MactavishCui MactavishCui deleted the taskApprovalNew branch January 10, 2025 07:12
@aiwenmo aiwenmo added the New Feature New feature label Jan 15, 2025
@aiwenmo aiwenmo added this to the 1.3.0 milestone Jan 15, 2025
@aiwenmo aiwenmo mentioned this pull request Jan 23, 2025
51 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Feature New feature
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants