Skip to content

learn move for starcoin#80

Merged
jolestar merged 12 commits intomovefuns:mainfrom
Tonyce:main
Aug 22, 2022
Merged

learn move for starcoin#80
jolestar merged 12 commits intomovefuns:mainfrom
Tonyce:main

Conversation

@Tonyce
Copy link
Copy Markdown
Contributor

@Tonyce Tonyce commented Aug 10, 2022

No description provided.

@auto-assign auto-assign Bot requested a review from pause125 August 10, 2022 06:01
@github-actions
Copy link
Copy Markdown

Welcome to the Web3 StarTrek 🎉!

You can take a look at the Starcoin Cookbook first, there is a lot of Starcoin knowledge in it.
If you are having trouble learning about Starcoin, you can ask and discuss in the development group.

@Tonyce
Copy link
Copy Markdown
Contributor Author

Tonyce commented Aug 14, 2022

@nkysg 我要毕业

Copy link
Copy Markdown
Collaborator

@uvd uvd left a comment

Choose a reason for hiding this comment

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

Can graduate

@nkysg
Copy link
Copy Markdown
Contributor

nkysg commented Aug 21, 2022

#61 参考这里相关描述,starcoin-cookbook的2个PR都是一个主题,认为是一个,dapp是一个,我认为还差一个 @Tonyce T。 两个人Approve就可以通过,还有另外reviewer一个Approve也可以。

@Tonyce
Copy link
Copy Markdown
Contributor Author

Tonyce commented Aug 21, 2022

@nkysg 这一个主题怎么说呢?那我是不是也可以理解我的两个 cookbook 的 PR 是三个主题呢。一个是 frontEnd,一个是 move contract,一个是合约的测试用例。考虑数量的同时,是不是也考虑下质量?让大家搞起形式主义来,就不大好了吧。

另修了一个小 bug :movefuns/movetool#54

@nkysg
Copy link
Copy Markdown
Contributor

nkysg commented Aug 21, 2022

@nkysg 这一个主题怎么说呢?那我是不是也可以理解我的两个 cookbook 的 PR 是三个主题呢。一个是 frontEnd,一个是 move contract,一个是合约的测试用例。考虑数量的同时,是不是也考虑下质量?让大家搞起形式主义来,就不大好了吧。

另修了一个小 bug :starcoinorg/dapps#54

这个提议又不是我提的,我只是提出我review观点,你若觉得有问题,可以找提出提议的人,其他人review也可 @Tonyce 。你这个starcoin-cookbook难道不是一个相关主题。我觉得提出提议的人是为了让开发者多熟悉相关内容。

@Tonyce
Copy link
Copy Markdown
Contributor Author

Tonyce commented Aug 21, 2022

@nkysg 这一个主题怎么说呢?那我是不是也可以理解我的两个 cookbook 的 PR 是三个主题呢。一个是 frontEnd,一个是 move contract,一个是合约的测试用例。考虑数量的同时,是不是也考虑下质量?让大家搞起形式主义来,就不大好了吧。
另修了一个小 bug :starcoinorg/dapps#54

这个提议又不是我提的,我只是提出我review观点,你若觉得有问题,可以找提出提议的人,其他人review也可 @Tonyce 。你这个starcoin-cookbook难道不是一个相关主题。我觉得提出提议的人是为了让开发者多熟悉相关内容。

那往大了说 cookbook 是不是就一个主题。🐶保命

刚刚加了小bug 的修复 😊 @nkysg

@nkysg
Copy link
Copy Markdown
Contributor

nkysg commented Aug 21, 2022

我觉得你那个starcoin-cookbook是合约相关开发的一个很好的教程,2次提交都是关于这个教程的,我认为是一个主题。你也可以提交其他starcoin-cookbook的不同类型问题,比如https://github.com/starcoinorg/starcoin-cookbook/issues 这里的,或者starcoin-framework相关,或者starcoin。我觉得都属于不同主题。

@Tonyce
Copy link
Copy Markdown
Contributor Author

Tonyce commented Aug 21, 2022

我觉得你那个starcoin-cookbook是合约相关开发的一个很好的教程,2次提交都是关于这个教程的,我认为是一个主题。你也可以提交其他starcoin-cookbook的不同类型问题,比如https://github.com/starcoinorg/starcoin-cookbook/issues 这里的,或者starcoin-framework相关,或者starcoin。我觉得都属于不同主题。

嗯。我刚刚有一个小的 bug 修复的 PR ,你看看满足条件了不 @nkysg

@Tonyce
Copy link
Copy Markdown
Contributor Author

Tonyce commented Aug 22, 2022

想以 web + move 毕业

@geometryolife
Copy link
Copy Markdown
Collaborator

@Tonyce 同意毕业,Cookbook 方面的贡献我觉得不错了。

@nkysg nkysg requested a review from geometryolife August 22, 2022 04:50
Comment thread members/tonyce.md Outdated
Comment on lines +1 to +6


# 星航计划加入申请表

> 以下是示例内容,请根据自己的情况修改。

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

多余的空白,注释删一下

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

hao

@Tonyce
Copy link
Copy Markdown
Contributor Author

Tonyce commented Aug 22, 2022

感谢 @uvd 对我的支持、指导和鼓励

@uvd
Copy link
Copy Markdown
Collaborator

uvd commented Aug 22, 2022

@jolestar

@jolestar
Copy link
Copy Markdown
Contributor

🎉毕业

curl 'http://main.seed.starcoin.org/' \
  -H 'content-type: application/json' \
  --data-raw '{"jsonrpc":"2.0","method":"contract.call_v2","params":[{"function_id":"0x00000000000000000000000000000001::PriceOracle::read","args":["0x82e35b34096f32c42061717c06e44a59"],"type_args":["0x00000000000000000000000000000001::STCUSDOracle::STCUSD"]}],"id":0}'
{"jsonrpc":"2.0","result":[101696],"id":0}
1000/0.101696 = 9833.2 STC

https://stcscan.io/main/transactions/detail/0x4aa5fc323983c3cc482682cafc7ed794812ff4a09e10490da1e18deda6cc6e15

@jolestar
Copy link
Copy Markdown
Contributor

@jolestar jolestar merged commit f39d299 into movefuns:main Aug 22, 2022
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.

5 participants