Skip to content

微信小程序三方平台代上传代码提交额外信息对象 新增 requiredPrivateInfos #3627

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

zhoushi1
Copy link

关于新增 requiredPrivateInfos 说明 关于地理位置接口新增与相关流程调整可以查看社区公告: 点击查看 7.14后,在代码中使用的地理位置相关接口(共计 8 个,见表1),第三方开发者均需要在 ext_json 参数中 requiredPrivateInfos 配置项中声明 在ext_json参数中配置requiredPrivateInfos,其规则为「整体替换」。即如果在 app.json 里也配置了,那么最终会是ext_json的配置会覆盖 app.json 配置的requiredPrivateInfos。其余规则可查看下方的「ext_json补充说明」 在ext_json参数中配置 requiredPrivateInfos 示例如下 { "template_id": "95", "ext_json": "{"requiredPrivateInfos":["onLocationChange","startLocationUpdate"]}", "user_version": "V1.0", "user_desc": "test" } requiredPrivateInfos主要会检查格式是否正确,填入的 api 名称是否正确,填入的 api 名称是否有权限,填入的 api 名称是否互斥。对应的错误码可查看文档末尾的错误码文档。 requiredPrivateInfos在2022.7.14后才会生效,文档提前更新是为了方便开发者可以提前了解接口的参数变更规则,提前进行调整。

@binarywang binarywang requested a review from Copilot June 27, 2025 09:59
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new requiredPrivateInfos field to support location-related permissions in the ext_json payload and enforces consistent brace usage in several utility methods.

  • Added requiredPrivateInfos array with accompanying JavaDoc.
  • Updated addExt, addExtPage, and addPage methods to always use braces for single-line if statements.
Comments suppressed due to low confidence (1)

weixin-java-open/src/main/java/me/chanjar/weixin/open/bean/ma/WxMaOpenCommitExtInfo.java:97

  • [nitpick] The Javadoc block from lines 97–113 is very verbose and duplicates external documentation. Consider shortening it to a brief summary and providing a link to the official doc for maintainability.
  /**

Copy link
Author

@zhoushi1 zhoushi1 left a comment

Choose a reason for hiding this comment

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

已确认

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.

1 participant