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

星航计划加入申请表-jzhao #111

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

jzhaopanghu
Copy link

现阶段已自学了一段时间move语言和starcoin、aptos公链,但是没有按照要求报名和整理学习日志、学习成果。
现在会整理一下日志和成功加到里面来。

@github-actions
Copy link

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.

@jzhaopanghu
Copy link
Author

@jolestar 王老师您好,学习总结和练习项目已整理完毕,现请求毕业。

@jolestar
Copy link
Contributor

Aptos 上的 球星卡 想法不错,不过实现略简单了。希望可以继续完善,比如用 NFT 表达,比如带上前端界面。

其他导师也看看。

最近 MoveFunsDAO 缺少资金支持,所以最后的奖励暂时没有了,可以等待后续新的激励计划。

@jzhaopanghu
Copy link
Author

Aptos 上的 球星卡 想法不错,不过实现略简单了。希望可以继续完善,比如用 NFT 表达,比如带上前端界面。

其他导师也看看。

最近 MoveFunsDAO 缺少资金支持,所以最后的奖励暂时没有了,可以等待后续新的激励计划。

好的 后续会继续完善一下 谢谢~

@yubing744
Copy link
Collaborator

Aptos 上的 球星卡 想法不错,不过实现略简单了。希望可以继续完善,比如用 NFT 表达,比如带上前端界面。

其他导师也看看。

最近 MoveFunsDAO 缺少资金支持,所以最后的奖励暂时没有了,可以等待后续新的激励计划。

我看了一下星球卡这个Demo,和真实的项目有些偏差,最好做个界面和钱包集成一下,这样能发现一些问题。如果做好了应该可以L1毕业。

@jzhaopanghu
Copy link
Author

好的 那我再完善一下 谢谢~

@jzhaopanghu
Copy link
Author

@yubing744 已添加前端项目(界面比较low,主要做后端的前端不太会..),因为不知道具体实际项目大概是什么样子,所以我就把球星卡demo的功能通过前端和钱包集成实现了一下,辛苦老师看下OTZ。

@yubing744
Copy link
Collaborator

@yubing744 已添加前端项目(界面比较low,主要做后端的前端不太会..),因为不知道具体实际项目大概是什么样子,所以我就把球星卡demo的功能通过前端和钱包集成实现了一下,辛苦老师看下OTZ。

我看了你的前端实现,有些问题:
1、代码中不能出现私钥,你需要集成钱包来获取交易授权
https://github.com/jzhaopanghu/football_dapp/blob/ff043116f9d024289e22e8b1c37a904eb54c67c8/src/App.tsx#L62

2、建议将球星卡的合约管理和用户通过合约购买球星卡拆分成两个页面

3、你似乎还没有理解什么是资源,我看你定义的FootbarStar还有 drop 能力
https://github.com/jzhaopanghu/football_demo/blob/fa9cce4e48c9b65a48aa5c7c5afeb4b53367a608/sources/football.move#L18

@jzhaopanghu
Copy link
Author

@yubing744 已去除私钥并将页面进行了拆分,drop确实有问题,疏忽了,resource只能有key和store,感谢指出问题,辛苦老师再看一下。

@yubing744
Copy link
Collaborator

@yubing744 已去除私钥并将页面进行了拆分,drop确实有问题,疏忽了,resource只能有key和store,感谢指出问题,辛苦老师再看一下。

我看了一下还是有一些问题:
1、球星卡没有做数量限制,任何人都可以 mint, 不能体现资源的稀缺性
2、安全漏洞:new_star 和 mint 方法标记为 public, hacker 可以绕过 create_star 方法直接 mint
https://github.com/jzhaopanghu/football_demo/blob/bb6002583c1eabd3376d92b66936c603eac804aa/sources/football.move#L34
3、前端调用合约报错
image

可以参考这里的例子再完善一下:
https://github.com/aptos-labs/aptos-core/tree/main/aptos-move/move-examples

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.

3 participants