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

增加sql上线可执行时间选择,且审核人可修改这个时间 #257

Closed
wants to merge 1 commit into from

Conversation

yzypals
Copy link
Collaborator

@yzypals yzypals commented Jun 6, 2019

增加sql上线可执行时间选择,且审核人可修改这个时间

@hhyo hhyo added the enhancement New feature or request label Jun 9, 2019
@yzypals yzypals closed this Jun 9, 2019
@yzypals yzypals deleted the etimesql branch June 9, 2019 12:57
Copy link
Owner

@hhyo hhyo left a comment

Choose a reason for hiding this comment

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

前面就review忘提交了

@@ -180,6 +180,8 @@ class SqlWorkflow(models.Model):
engineer_display = models.CharField('发起人中文名', max_length=50, default='')
status = models.CharField(max_length=50, choices=SQL_WORKFLOW_CHOICES)
audit_auth_groups = models.CharField('审批权限组列表', max_length=255)
starttime = models.DateTimeField('可执行起始时间', null=True, blank=True)
Copy link
Owner

Choose a reason for hiding this comment

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

这两个字段名称可以推敲一下,我也想不到好的😄

@@ -69,6 +75,22 @@ def can_execute(user, workflow_id):
result = True
return result

def on_Correct_time_period(workflow_id):
Copy link
Owner

Choose a reason for hiding this comment

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

函数名全部小写

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

嗯 , 我修改下 重新提, 昨天 看好久没动静 就撤掉了

@LeoQuote
Copy link
Collaborator

建议不要关闭pr,直接在分支上推送,一旦关闭在代码基础上的讨论就都丢失了。看不到演化过程。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants