-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
工单部分接口调用合并至api #1897
base: master
Are you sure you want to change the base?
工单部分接口调用合并至api #1897
Conversation
hhyo
commented
Oct 7, 2022
- 升级部分基础依赖(未完全测试)
- 调整api项目的目录结构
- 部分工单接口转移至api,版本为v2
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #1897 +/- ##
==========================================
- Coverage 76.34% 76.05% -0.30%
==========================================
Files 115 119 +4
Lines 15852 15862 +10
==========================================
- Hits 12103 12064 -39
- Misses 3749 3798 +49 ☔ View full report in Codecov by Sentry. |
e7d2cec
to
cf186c6
Compare
b69ccc8
to
a107ec6
Compare
3ce6ab8
to
6f6a816
Compare
6f6a816
to
a43e2b0
Compare
@@ -27,7 +27,7 @@ | |||
) | |||
from sql.utils.resource_group import auth_group_users | |||
from sql.utils.workflow_audit import Audit | |||
from sql_api.serializers import ( | |||
from sql_api.serializers.serializers import ( |
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.
可以在 serializers/__init__.py
里面 from .sql_workflow import xxxx
这样之前 import 的不用改, 这个路径也不会显得很奇怪, 你觉得如何