-
Notifications
You must be signed in to change notification settings - Fork 1.3k
feat: 【bpm】bpmn设计器: 业务表单流程添加重新发起功能 #209
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
Conversation
src/views/bpm/oa/leave/index.vue
Outdated
| v-hasPermi="['bpm:oa-leave:create']" | ||
| link | ||
| type="primary" | ||
| @click="handleReInitiate(scope.row)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
单词用 recreate 更好哈。
src/views/bpm/oa/leave/index.vue
Outdated
| router.push({ | ||
| name: 'OALeaveCreate', | ||
| query: { | ||
| id: row.id, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
就传递 oldId 噶。
然后,processInstanceId 不传递。因为 oldId 可以查询到流程编号噶。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
看到了,嘿嘿,直接用oldId从后台拿到全部业务数据了,我改下
src/views/bpm/oa/leave/create.vue
Outdated
| const getLeaveData = async (id: number) => { | ||
| formLoading.value = true | ||
| const data = await LeaveApi.getLeave(id) | ||
| if(!data){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if (!data) {
代码排版注意下哈。
| defineOptions({ name: 'BpmOALeaveCreate' }) | ||
| const message = useMessage() // 消息弹窗 | ||
| const { delView } = useTagsViewStore() // 视图操作 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
另外:流程审批界面那,应该也有个重新发起。可以也加下哈。
| } else if (processDefinitionDetail.formType === 10) { | ||
| //如果是【流程表单】,跳转到流程发起界面 | ||
| await router.push({ | ||
| name: 'BpmProcessInstanceDynamicFormCreate', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
name: 'BpmProcessInstanceCreate',
这里不用改呀。
| await router.push({ | ||
| path: processDefinitionDetail.formCustomCreatePath, | ||
| query: { | ||
| id: row.businessKey |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
最好传递 oldId ;别的地方也改下哈。
YunaiV
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
第二轮 review
No description provided.