-
Notifications
You must be signed in to change notification settings - Fork 42
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
DEX 开发课程(Wtfswap)任务认领 #59
Comments
前端部分的架构设计可以参考章节 https://github.com/WTFAcademy/WTF-Dapp/blob/main/P201_InitFrontend/readme.md 设计稿图片在 https://github.com/WTFAcademy/WTF-Dapp/blob/main/P201_InitFrontend/readme.md 需要 sketch 设计稿源文件的可以联系我 |
另外说明一下,前端部分使用 Ant Design 和 Ant Design Web3 的组件开发,如果发现设计稿的样式和组件不一致的地方以组件为准,这样尽量避免代码中出现太多自定义样式和 hack 代码,保持教程的简洁,有不明确的地方可以讨论确认。 |
我来! 我先看看教程 |
👍 可以先帮忙改改错字啥的 |
发一下设计稿给我吧, 可以动手了. 我先开发一下pool吧 |
设计稿发你邮箱 [email protected] 了,不过我看 #63 @Lumend 已经认领了,要和他对一下,避免重复开发 |
认领factory |
#82 🫡 这个 issue |
认领 PoolManager.sol |
|
认领 SwapRouter.sol 合约开发 |
认领 PositionManager.sol 合约开发 |
|
Pool swap 交易接口开发 P107 |
#81 认领 Pool LP 相关功能开发 |
|
所有任务已经认领完成,感谢大家,可以各自投入开发了,除了代码,还有开发过程中的课程哈。有问题随时讨论沟通。 |
大家提交合约代码的时候,代码主要先往 https://github.com/WTFAcademy/WTF-Dapp/tree/main/demo-contract 这里提,保证工程可以跑起来,等最后 review 没有问题合并了,最后再补充课程以及需要单独放到每一讲课程中的代码。 |
如何参与贡献
下面的课程列表都已经有社区的同学认领了,但是因为大家时间问题,进度比较慢。为了加快进度,欢迎更多的同学直接参与贡献。如果对课程感兴趣,你可以直接提 PR 补充课程内容,可以是一个还未实现的合约的方法,也可以是一个错别字,任何贡献都可以。
如果担心你正在处理的内容已经有其他社区同学已经再做,你可以先提个对应的 ISSUE 指给自己,如果其他同学有异议可以直接在 ISSUE 下面讨论。或者你也可以到其他同学创建的对应的 ISSUE 下面确认是否可以“抢”过某一部分开发任务。当然,即便有冲突也没关系,大家也可以一起帮忙 review 就行,谁先提交了合适的 PR 就优先合并谁的,保持开放的态度。
课程列表
认领方法
如下图所示:
通过该 ISSUE 任务后面的 icon 点击新建 ISSUE 并 assign 给自己即可(如果没有权限可以在这个 issue 下面留言,我这边来处理),然后提交 PR 来参与课程建设。所有课程计划在 6 月底之前完成开发,其中前端逻辑的开发可能部分依赖于 UI 的开发,所以 UI 的开发尽量要更早完成,你也可以选择认领 UI + 逻辑整个一部分功能的开发。
大家提交合约代码的时候,代码主要先往 https://github.com/WTFAcademy/WTF-Dapp/tree/main/demo-contract 这里提,保证工程可以跑起来,等最后 review 没有问题合并了,最后再补充课程以及需要单独放到每一讲课程中的代码。
The text was updated successfully, but these errors were encountered: