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.

Sign up for GitHub

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

Jump to bottom

feat: added operation audit for data source #1962

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

feat: added operation audit for data source #1962

wants to merge 11 commits into from
+272 −3

Conversation

rolin999
Copy link
Collaborator

Description

Fixes #1732 操作审计 app 模块 + 数据源 api 操作审计

Checklist

  • 填写 PR 描述及相关 issue (write PR description and related issue)
  • 代码风格检查通过 (code style check passed)
  • PR 中包含单元测试 (include unit test)
  • 单元测试通过 (unit test passed)
  • 本地开发联调环境验证通过 (local development environment verification passed)

@rolin999 rolin999 self-assigned this Oct 14, 2024
@rolin999 rolin999 added the doing 已经接受处理，正在开发实施 label Oct 14, 2024
narasux
narasux requested changes Oct 15, 2024
View reviewed changes
Copy link
Collaborator

@narasux narasux left a comment

Choose a reason for hiding this comment

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

有一些底层设计需要多讨论下，目前实现的不太合理

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@narasux narasux narasux requested changes

@nannan00 nannan00 Awaiting requested review from nannan00

Requested changes must be addressed to merge this pull request.

Assignees

@rolin999 rolin999

Labels
doing 已经接受处理，正在开发实施
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

用户管理审计日志
2 participants
@rolin999 @narasux