Skip to content

fix(user): correct comment typo for BusinessWritePermission default v…#615

Merged
iwanghc merged 1 commit into
mainfrom
dms/feat-813
May 12, 2026
Merged

fix(user): correct comment typo for BusinessWritePermission default v…#615
iwanghc merged 1 commit into
mainfrom
dms/feat-813

Conversation

@iwanghc
Copy link
Copy Markdown
Collaborator

@iwanghc iwanghc commented May 12, 2026

User description

…alue from 'flase' to 'false'

关联的 issue

https://github.com/actiontech/dms-ee/issues/813

描述你的变更

修复注释单词拼写错误

确认项(pr提交后操作)

Tip

请在指定复审人之前,确认并完成以下事项,完成后✅


  • 我已完成自测
  • 我已记录完整日志方便进行诊断
  • 我已在关联的issue里补充了实现方案
  • 我已在关联的issue里补充了测试影响面
  • 我已确认了变更的兼容性,如果不兼容则在issue里标记 not_compatible
  • 我已确认了是否要更新文档,如果要更新则在issue里标记 need_update_doc


Description

  • 修正业务写权注释拼写错误

  • 确保默认值描述与代码一致


Diagram Walkthrough

flowchart LR
  A["\"internal/dms/biz/user.go\""] -- "更新注释拼写" --> B["\"internal/dms/storage/model/model.go\""]
Loading

File Walkthrough

Relevant files
Bug fix
user.go
更新用户逻辑注释拼写                                                                                             

internal/dms/biz/user.go

  • 更正非系统管理员逻辑中注释:将 "flase" 改为 "false"
+1/-1     
model.go
更新数据模型注释拼写                                                                                             

internal/dms/storage/model/model.go

  • 修正数据模型中注释错误:将 "flase" 改为 "false"
+1/-1     

@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions
Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@iwanghc iwanghc merged commit 4161a21 into main May 12, 2026
1 check passed
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