Skip to content

refactor: 能力开放->应用管理,添加数据权限访问控制 - #215

Merged
Charles7c merged 1 commit into
continew-org:devfrom
luoqiz:refact-appmapper
Jul 20, 2026
Merged

refactor: 能力开放->应用管理,添加数据权限访问控制#215
Charles7c merged 1 commit into
continew-org:devfrom
luoqiz:refact-appmapper

Conversation

@luoqiz

@luoqiz luoqiz commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

PR 类型

  • 新 feature
  • Bug 修复
  • 功能增强
  • 文档变更
  • 代码样式变更
  • 重构
  • 性能改进
  • 单元测试
  • CI/CD
  • 其他

PR 目的

解决方案

PR 测试

Changelog

模块 Changelog Related issues

其他信息

提交前确认

  • PR 代码经过了完整测试,并且通过了代码规范检查
  • 已经完整填写 Changelog,并链接到了相关 issues
  • PR 代码将要提交到 dev 分支

Summary by CodeRabbit

  • 功能优化
    • 应用数据查询现已支持数据权限控制,提升数据访问的安全性与合规性。

@coderabbitai

coderabbitai Bot commented Jul 19, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

AppMapper移除BaseMapper依赖,改为继承DataPermissionMapper,从而更新其公开映射器契约并接入数据权限能力。

Changes

应用映射器数据权限

Layer / File(s) Summary
映射器数据权限契约
continew-plugin/continew-plugin-open/src/main/java/top/continew/admin/open/mapper/AppMapper.java
新增DataPermissionMapper导入,并将AppMapper的继承关系从BaseMapper改为DataPermissionMapper。

Estimated code review effort: 1 (Trivial) | ~5 minutes

Poem

兔子挥挥小耳朵,
映射器换上权限衣。
Base悄悄跳出门,
DataPermission来相依。
App数据更有序,
胡萝卜也安心。

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 标题概括了从“能力开放”重构到“应用管理”并新增数据权限访问控制,和变更目标一致。
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Charles7c
Charles7c merged commit d5eeaaf into continew-org:dev Jul 20, 2026
3 of 4 checks passed
@luoqiz
luoqiz deleted the refact-appmapper branch July 20, 2026 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants