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

Release neo3 #945

Merged
merged 14 commits into from
Nov 5, 2019
Merged

Release neo3 #945

merged 14 commits into from
Nov 5, 2019

Conversation

ShawnYun
Copy link
Contributor

Update fee and RPC-API

Copy link
Member

@superboyiii superboyiii left a comment

Choose a reason for hiding this comment

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

Test

Copy link
Member

@superboyiii superboyiii left a comment

Choose a reason for hiding this comment

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

Need fix.

Comment on lines 29 to 32
* **默认打开钱包:** 如果要调用与钱包相关的 API,也需要先修改配置文件 `config.json`,将:
- unlockwallet 改为 true 的状态,
- 并填写对象钱包的文件名和密码,如下所示:
```json
Copy link
Member

Choose a reason for hiding this comment

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

请修改:

  • 默认打开钱包: 如果设置启动neo-cli时自动打开钱包,也需要先修改配置文件。。。

docs/zh-cn/reference/rpc/latest-version/api/getblock.md Outdated Show resolved Hide resolved
docs/zh-cn/reference/rpc/latest-version/api/getversion.md Outdated Show resolved Hide resolved
Comment on lines +1 to +33
# getwalletheight 方法

获取当前钱包索引高度

> 执行此命令前需要在 Neo-CLI 节点中打开钱包。
>
> 此方法由插件提供,需要安装 [RpcWallet](https://github.com/neo-project/neo-plugins/releases) 插件才可以调用



## 调用示例

请求正文:

```json
{
"jsonrpc": "2.0",
"method": "getwalletheight",
"params": [],
"id": 1
}
```

响应正文:

```json
{
"jsonrpc": "2.0",
"id": 1,
"result": 2713183
}
```

Copy link
Member

Choose a reason for hiding this comment

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

此api已不存在,需要删除

Copy link
Member

@superboyiii superboyiii left a comment

Choose a reason for hiding this comment

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

sendtoaddress 和 sendmany 需要改一下,其他的没问题了。

Comment on lines 15 to 16
"params": [outputs_array,fee, change_address],
"id": 1
Copy link
Member

Choose a reason for hiding this comment

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

应改为 "params": [outputs_array]

Comment on lines 15 to 16
"params": [asset_id,address,value, fee, change_address],
"id": 1
Copy link
Member

Choose a reason for hiding this comment

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

应改为 "params": [asset_id,address,value],

Copy link
Member

@superboyiii superboyiii left a comment

Choose a reason for hiding this comment

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

Need a little fix.

Comment on lines 112 to 114
| [sendfrom](/api/rpcwallets/sendfrom.md) | \<asset_id>\<from>\<to>\<value>\[fee]\[change_address] | 从指定地址,向指定地址转账 | |
| [sendmany](/api/rpcwallets/sendmany.md) | \<outputs_array>\[fee]\[change_address] | 批量转账命令 | |
| [sendtoaddress](/api/rpcwallets/sendtoaddress.md) | \<asset_id>\<address>\<value>\[fee]\[change_address] | 向指定地址转账 | |
Copy link
Member

Choose a reason for hiding this comment

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

| sendfrom | <asset_id><from><to><value> | 从指定地址,向指定地址转账 | |
| sendmany | <outputs_array> | 在一笔交易中向指定地址发起多比转账| |
| sendtoaddress | <asset_id><address><value>| 向指定地址转账 | |


若要通过 HTTPS 的方式访问 RPC 服务器,需要在启动节点前修改配置文件 `config.json`,并设置域名、证书和密码:
### **配置 HTTPS**
Copy link
Member

Choose a reason for hiding this comment

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

这个一部分,感觉适合放到节点那一部分?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

原来的文档也是这么写的,要改到节点去吗

Copy link
Member

Choose a reason for hiding this comment

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

我们商量了一下,放到节点那边比较合适,关于配置的这一小块可以去除。

@Celia18305 Celia18305 merged commit 209f9ed into neo-project:release-neo3 Nov 5, 2019
ShawnYun added a commit to ShawnYun/docs that referenced this pull request Nov 6, 2019
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.

None yet

4 participants