Add miniprogram setting client #1388
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
overtrue
reviewed
Oct 31, 2018
| * @param int $bottom 剪裁框右下角y坐标（取值范围：[0, 1]） | ||
| */ | ||
| public function modifyHeadImage( | ||
| string $media_id, |
| * @param int $right 剪裁框右下角x坐标（取值范围：[0, 1]） | ||
| * @param int $bottom 剪裁框右下角y坐标（取值范围：[0, 1]） | ||
| */ | ||
| public function modifyHeadImage( |
| * | ||
| * @param string $signature 功能介绍（简介） | ||
| */ | ||
| public function modifySignature($signature) |
| * | ||
| * @param array $categories 类目数组 | ||
| */ | ||
| public function addCategory(array $categories) |
| * @param int $first_id 一级类目ID | ||
| * @param int $second_id 二级类目ID | ||
| */ | ||
| public function deleteCategory(int $first_id, int $second_id) |
| */ | ||
| public function setNickname( | ||
| string $nickname, | ||
| string $idcard = '', |
| public function setNickname( | ||
| string $nickname, | ||
| string $idcard = '', | ||
| string $license = '', |
| string $nickname, | ||
| string $idcard = '', | ||
| string $license = '', | ||
| array $stuff_list = [] |
| * | ||
| * @param int $audit_id 审核单id | ||
| */ | ||
| public function queryNickname($audit_id) |
| * | ||
| * @param string $nickname 名称（昵称） | ||
| */ | ||
| public function checkWxVerifyNickname($nickname) |
There was a problem hiding this comment.
isAvailableNickname($nickname)
mingyoung
reviewed
Oct 31, 2018
| */ | ||
| public function getBasicInfo() | ||
| { | ||
| return $this->httpPostJson('cgi-bin/account/getaccountbasicinfo'); |
There was a problem hiding this comment.
这个有考虑过，但该接口出现在开放平台文档的“代小程序实现业务”板块下，故对该接口的作用范围（即是否可以用于公众号等）并不确定，以致按照文档逻辑与其他小程序设置接口置于同处。希望可以获知更多相关信息。
There was a problem hiding this comment.
可以放在 OpenPlatform/Authorizer/MiniProgram/Account 下，参照公众号的 Account，这是继承公用的 Account 的:
https://github.com/overtrue/wechat/blob/master/src/OpenPlatform/Authorizer/OfficialAccount/Account/Client.php
There was a problem hiding this comment.
有此先例，那便可行。不过从接口对应URL来看，属于cgi-bin/account路径下的接口有getBasicInfo、updateAvatar、updateSignature，按你建议，是否需要将这三个接口一并移至新的Client下呢？
|
@overtrue 个人对overtrue/wechat的编码规范不够熟悉，造成了疏忽，感谢详细的指正。所有命名修改建议均已修改提交。 另外，新的提交中包含对测试脚本中原有错误的修复。对于提交PR之前没有运行测试检查的行为深感抱歉！ |
|
@mingyoung 现在 OK 吗？ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
对应Issue：#1262
添加开放平台的小程序信息设置系列接口。