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

🐛 修复 Role 获取的用户角色不正确的问题 #6

Merged
merged 1 commit into from
Aug 23, 2023
Merged

🐛 修复 Role 获取的用户角色不正确的问题 #6

merged 1 commit into from
Aug 23, 2023

Conversation

A-kirami
Copy link
Owner

这个 PR 带来了什么样的更改？

  • 错误修复
  • 新功能
  • 文档/注释
  • 代码格式
  • 代码重构
  • 测试用例
  • 性能优化
  • 外观样式
  • 项目构建
  • 依赖环境
  • 持续集成/部署
  • 其他，请描述:

这个 PR 是否存在破坏性变更？

  • 是的，并已在 issue #___ 号中获得批准
  • 没有

描述

将 Role 方法中的 user_id 参数类型改为 str

动机和背景

获取用户角色时因为 Role 的部分方法没有将参数中的 user_id 转换为 str，导致获取的结果不正确

其他信息

检查工作

  • 我对我的代码进行了注释，特别是在难以理解的部分
  • 我的更改需要更新文档，并且已对文档进行了相应的更改
  • 我添加了测试并且已经在本地通过，以证明我的修复补丁或新功能有效
  • 我已检查并确保更改没有与其他打开的 Pull Requests 重复

@A-kirami A-kirami added the bug Something isn't working label Aug 23, 2023
@A-kirami A-kirami merged commit 31ab813 into main Aug 23, 2023
3 checks passed
@A-kirami A-kirami deleted the fix/userid-type branch August 23, 2023 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@A-kirami