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.

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

[feat] 根据uuid查询用户信息 #24

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

xiangZr-hhh
Copy link

@xiangZr-hhh xiangZr-hhh commented Jun 5, 2024

🤔 这个变动的性质是?

  • ✨ 新特性开发
  • 🐞 Bug 修复
  • 🦢 重构
  • 📝 文档、注释
  • 🚀 打包、CI相关
  • 🖐️ 测试用例
  • 📘 其他改动(是关于什么的改动?)

💡 需求背景和解决方案

⚠️ 是否是破坏性的改动?

  • 是(yes)

  • 否(no)

xiangZr-hhh added 4 commits June 4, 2024 17:57
用户公开信息增加了角色id、会员id字段
用户公开信息增加了角色id、会员id字段
用户公开信息增加了角色id、会员id字段
@xiangZr-hhh xiangZr-hhh requested a review from XiaoLFeng June 5, 2024 01:26
@xiangZr-hhh xiangZr-hhh closed this Jun 5, 2024
@xiangZr-hhh xiangZr-hhh reopened this Jun 6, 2024
xiangZr-hhh added 4 commits June 7, 2024 08:08
修正了部分代码规范问题
修正了部分代码规范问题
修正了部分代码规范问题
修正了部分代码规范问题
Copy link
Member

@XiaoLFeng XiaoLFeng left a comment

Choose a reason for hiding this comment

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

修改后拉取 master 更新后提交

}
UserPublicInfoVO userVO = userService.getMyUserProfileInfo(uuid);
return ResultUtil.success("获取当前用户信息成功", userVO);
}

Copy link
Member

Choose a reason for hiding this comment

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

该部分内容不可删除

redisUtil.setHasKey(redisKey, Util.INSTANCE.objectToMap(userPublicInfoVO));

return userPublicInfoVO;
}

Copy link
Member

Choose a reason for hiding this comment

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

该部分内容不可删除

@@ -213,4 +214,53 @@ object Util {
}
return target.getDeclaredConstructor().newInstance()
}

Copy link
Member

Choose a reason for hiding this comment

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

有 BeanUtils 工具类,包括了这一个方法

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