已合并
add OH_OsAccount_GetNameByLocalId #3271
add OH_OsAccount_GetNameByLocalId #3271
已合并
yeyuning创建于 3月27日
yeyuning
yeyuning
3月27日

一、内容说明(相关的Issue)

https://gitcode.com/openharmony-sig/interface_native_header/issues/2850

二、建议测试周期和提测地址

建议测试完成时间:xxxx.xx.xx
投产上线时间:xxxx.xx.xx
提测地址:CI环境/压测环境
测试账号:

三、变更内容

  • 3.1 关联PR列表
    https://gitcode.com/openharmony/interface_sdk_c/pull/4792

  • 3.2 数据库和部署说明

    1. 常规更新
    2. 重启unicorn
    3. 重启sidekiq
    4. 迁移任务:是否有迁移任务,没有写 "无"
    5. rake脚本:bundle exec xxx RAILS_ENV = production;没有写 "无"
  • 3.4 其他技术优化内容(做了什么,变更了什么)

    • 重构了 xxxx 代码
    • xxxx 算法优化
  • 3.5 废弃通知(什么字段、方法弃用?)

  • 3.6 后向不兼容变更(是否有无法向后兼容的变更?)

四、研发自测点(自测哪些?冒烟用例全部自测?)

自测测试结论:

五、测试关注点(需要提醒QA重点关注的、可能会忽略的地方)

检查点:

需求名称 是否影响xx公共模块 是否需要xx功能 需求升级是否依赖其他子产品
xxx 需要 不需要

接口测试:

性能测试:

并发测试:

其他:

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 yeyuning 的贡献)
yeyuningyeyuning
3月27日 关联了issue:add OH_OsAccount_GetNameByLocalId
openharmony_ci
openharmony_ci成员
3月27日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
openharmony_ciopenharmony_ci成员
3月27日 添加了label:dco检查成功
x_huahua
x_huahua3月27日进行代码检视2
zh-cn/native_sdk/account/os_account.h
已过期
@@ -60,0 +61,4 @@
61+ * @param localId 目标系统帐号的本地ID。
62+ * @param name 名称字符数组,该数组应为名称和终止字符(“\0”)留出空间。名称的最大大小为 LOGIN_NAME_MAX。
63+ * @param name_size 名称字符数组的大小。
64+ * @return OS_ACCOUNT_ERR_OK:表示成功;<br>
x_huahua
x_huahua3月27日评论:
改动建议
64
- * @return OS_ACCOUNT_ERR_OK:表示成功;<br>
64
+ * @return OS_ACCOUNT_ERR_OK:表示成功;\n
应用建议

都用\n换行

likedislike
System
系统消息系统
3月27日 评论:

changed this line on d4fea5f9 view diff detail

yeyuningyeyuning
3月27日 修改了pull request 的描述
x_huahua
x_huahua3月27日进行代码检视2
zh-cn/native_sdk/account/os_account.h
已过期
@@ -60,0 +59,4 @@
59+ *
60+ * @permission ohos.permission.GET_LOCAL_ACCOUNT_IDENTIFIERS
61+ * @param localId 目标系统帐号的本地ID。
62+ * @param name 名称字符数组,该数组应为名称和终止字符(“\0”)留出空间。名称的最大大小为 LOGIN_NAME_MAX。
x_huahua
x_huahua3月27日评论:
改动建议
62
- * @param name 名称字符数组,该数组名称终止符(“\0”)留出空间。名称的最大大小 LOGIN_NAME_MAX。
62
+ * @param name 名称字符数组应包含名称终止符(“\0”)最大长度为LOGIN_NAME_MAX。
应用建议

咋理解LOGIN_NAME_MAX?这个概念在哪里介绍过?业内专业名词?

likedislike
System
系统消息系统
3月31日 评论:

changed this line on 7e718e92 view diff detail

yeyuningyeyuning
3月27日 强制推送  1 个提交:d4fea5f9-add OH_OsAccount_GetNameByLocalId
x_huahua
x_huahua3月27日进行代码检视2
zh-cn/native_sdk/account/os_account.h
@@ -60,0 +60,4 @@
60+ * @permission ohos.permission.GET_LOCAL_ACCOUNT_IDENTIFIERS
61+ * @param localId 目标系统帐号的本地ID。
62+ * @param name 名称字符数组,该数组应为名称和终止字符(“\0”)留出空间。名称的最大大小为 LOGIN_NAME_MAX。
63+ * @param name_size 名称字符数组的大小。
x_huahua
x_huahua3月27日评论:

有取值范围嘛?

likedislike
yeyuning
yeyuning
3月31日 评论:

已体现在name中

openharmony_ci
openharmony_ci成员
3月27日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
x_huahua
x_huahua3月27日进行代码检视2
zh-cn/native_sdk/account/os_account.h
已过期
@@ -60,0 +59,4 @@
59+ *
60+ * @permission ohos.permission.GET_LOCAL_ACCOUNT_IDENTIFIERS
61+ * @param localId 目标系统帐号的本地ID。
62+ * @param name 名称字符数组,该数组应为名称和终止字符(“\0”)留出空间。名称的最大大小为 LOGIN_NAME_MAX。
x_huahua
x_huahua3月27日评论:

这块引号到底是中文的还是英文的?咋和67行不一致?

likedislike
System
系统消息系统
3月31日 评论:

changed this line on 7e718e92 view diff detail

x_huahua
x_huahua3月27日进行代码检视2
zh-cn/native_sdk/account/os_account.h
已过期
@@ -57,0 +66,3 @@
66+ * OS_ACCOUNT_ERR_INTERNAL_ERROR:表示内部错误;\n
67+ * OS_ACCOUNT_ERR_INVALID_PARAMETER:表示name为空指针或名称长度(包括结束符'\0')大于name_size;\n
68+ * OS_ACCOUNT_ERR_ACCOUNT_NOT_FOUND:表示未找到帐号;\n
69+ * OS_ACCOUNT_ERR_RESTRICTED_ACCOUNT:表示帐号受限,无法查询;
x_huahua
x_huahua3月27日评论:

此条代码评论区间+66+69

补充os_account_common.h修改,不然你这有的错误码搜不到啊

likedislike
System
系统消息系统
3月31日 评论:

changed this line on be0eddcb view diff detail

yeyuningyeyuning
3月31日 强制推送  1 个提交:7e718e92-add OH_OsAccount_GetNameByLocalId
openharmony_ci
openharmony_ci成员
3月31日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
yeyuningyeyuning
3月31日 强制推送  1 个提交:751a84a9-add OH_OsAccount_GetNameByLocalId
openharmony_ci
openharmony_ci成员
3月31日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
yeyuningyeyuning
3月31日 强制推送  1 个提交:be0eddcb-add OH_OsAccount_GetNameByLocalId
openharmony_ci
openharmony_ci成员
3月31日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
yeyuning
yeyuning
3月31日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
3月31日 评论:

首次触发
门禁构建开始,包含静态检查,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/69cbcc3b64650f998b75aed0/runlist

likedislike
openharmony_ciopenharmony_ci成员
3月31日 添加了label:静态检查失败
openharmony_ci
openharmony_ci成员
3月31日 评论:

代码门禁未通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/69cbcc3b64650f998b75aed0/runlist

静态检查:

# check type result report
1 codeCheck noPass >>>

likedislike
时睿时睿成员
4月7日 通过审查
yeyuning
yeyuning
4月7日 评论:

start build

likedislike
openharmony_ciopenharmony_ci成员
4月7日 删除了label:静态检查失败
openharmony_ci
openharmony_ci成员
4月7日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/69d4773864650f998b867d29/runlist

likedislike
openharmony_ciopenharmony_ci成员
4月7日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
4月7日 通过测试
openharmony_ci
openharmony_ci成员
4月7日 评论:

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/69d4773864650f998b867d29/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

likedislike
openharmony_ciopenharmony_ci成员
4月7日 关闭了关联的issue
openharmony_ciopenharmony_ci成员
4月7日 合入了pull request,合并节点 SHA:b59c1465442ae083be8c175362904d0d9c4c93d6