-
Notifications
You must be signed in to change notification settings - Fork 498
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
feat:流水线编辑页面,增加更多操作 #10784 #10994
feat:流水线编辑页面,增加更多操作 #10784 #10994
Conversation
# Reviewed, transaction id: 18876
@@ -63,16 +63,44 @@ | |||
> | |||
<span class="debug-pipeline-draft-btn"> | |||
{{ $t("debug") }} |
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.
这里去掉了后面的入口,没必要再弄个span标签包起来了,可以去掉,还有,对应的css也要去掉,要不然很多无用的代码在里面
{{ $t('close') }} | ||
</bk-button> | ||
</footer> | ||
</bk-dialog> |
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.
这里我记得有一个versionDiffEntry的组件可以复用的,不需要自己再去重复写
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.
这个组件我看到了,但是它还有其他的一些东西,比如头部的版本选择,而且它有个文字按钮。我这里只需要对比当前草稿和当前最新版本,只需要yaml-diff这个组件,所以就没有用versionDiffEntry组件了
]) | ||
]) | ||
}, | ||
async diffVersionConfirm () { |
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.
diffversion?? 这里应该是删除吧
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.
这里删除草稿有两种情况,一种是有草稿但是是那种新建的流水线没操作过的,这种删除草稿就把该条流水线删除了,另一种就是单独的删除当前草稿。我单独把两种情况写了处理方法,在删除的时候分别调用了
…o issue_10784 # Reviewed, transaction id: 19137
# Reviewed, transaction id: 19214
# Reviewed, transaction id: 19216
# Reviewed, transaction id: 19293
# Reviewed, transaction id: 19302
# Reviewed, transaction id: 19316
#10784