[refactor]: (SqlExecWorkflow) List and Create Module #312
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
我没啥意见。这pipeline要修么？还是合完再修？ |
可以先合并，这个和进去后我开下 detail 部分的 pr。然后再一起处理， detail 部分调整了这个 pr 的部分改动。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
issue 地址：actiontech/sqle#2429
重构 SQL上线工单的 List 与 Create 模块
放弃了 redux，还是使用了 props 来处理数据流
暂时还没走单元测试。后面补。
重构相关的变更后面会在重构结束后补充文档。修复了一些 bug 以及想要优化一些操作，待和产品沟通确认
assign in @Rain-1214