-
Notifications
You must be signed in to change notification settings - Fork 44
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
feat: Pool LP 相关功能开发 V2 #104
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
好的,收到
愚指导 ***@***.***> 于2024年7月30日周二 16:20写道:
… @yutingzhao1991 <https://github.com/yutingzhao1991> requested your review
on: #104 <#104> feat: Pool LP
相关功能开发 V2.
—
Reply to this email directly, view it on GitHub
<#104 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYSDDWWN5S7E3QKMBTN64MDZO5EERAVCNFSM6AAAAABLTZEFISVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGY4TEMBRGI2DOOI>
.
You are receiving this because your review was requested.Message ID:
***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
学习并看完了本次review涉及到的代码( pool 和 Ipool 合约), 修改了笔误,请问 mint 等函数增加 data 参数是需要增加 UniswapV3Callback 的功能吗
嗯,mint 成功之后应该是要调用一下 callback 的。这块我也在看,可以先实现必要功能,对于课程来说,如果不是特别必要的,后面也可以考虑删除。另外 @hattiepwvi 这个 PR 你继续跟进来改吗? |
好的,收到, 有机会的话,根据时间安排, 我非常开心可以继续跟进修改。
愚指导 ***@***.***> 于2024年7月31日周三 09:34写道:
… 学习并看完了本次review涉及到的代码( pool 和 Ipool 合约), 修改了笔误,请问 mint 等函数增加 data 参数是需要增加
UniswapV3Callback 的功能吗
嗯,mint 成功之后应该是要调用一下 callback
的。这块我也在看,可以先实现必要功能,对于课程来说,如果不是特别必要的,后面也可以考虑删除。另外 @hattiepwvi
<https://github.com/hattiepwvi> 这个 PR 你继续跟进来改吗?
—
Reply to this email directly, view it on GitHub
<#104 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYSDDWR72LFLIDA634SUWTLZPA5K7AVCNFSM6AAAAABLTZEFISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJZGQ3DKNBRGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
b496856
to
da0baa1
Compare
嗯,可以的,我们可以一起来搞,我也参与来一起看看。我更新了一下 feat/lpinpool 这个,现在把 feat/lpinpool 这个设置为保护分支了,后面我们都单独提 PR 到这个分支吧。我先写写 |
好的,收到,我先看看 feat/lpinpool 保护分支,期待拜读。
愚指导 ***@***.***> 于2024年8月1日周四 13:52写道:
… 好的,收到, 有机会的话,根据时间安排, 我非常开心可以继续跟进修改。 愚指导 *@*.
*> 于2024年7月31日周三 09:34写道: … <#m_-1557853901238232692_>
学习并看完了本次review涉及到的代码( pool 和 Ipool 合约), 修改了笔误,请问 mint 等函数增加 data 参数是需要增加
UniswapV3Callback 的功能吗 嗯,mint 成功之后应该是要调用一下 callback
的。这块我也在看,可以先实现必要功能,对于课程来说,如果不是特别必要的,后面也可以考虑删除。另外 @hattiepwvi
<https://github.com/hattiepwvi> https://github.com/hattiepwvi
<https://github.com/hattiepwvi> 这个 PR 你继续跟进来改吗? — Reply to this email
directly, view it on GitHub <#104 (comment)
<#104 (comment)>>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYSDDWR72LFLIDA634SUWTLZPA5K7AVCNFSM6AAAAABLTZEFISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJZGQ3DKNBRGE
<https://github.com/notifications/unsubscribe-auth/AYSDDWR72LFLIDA634SUWTLZPA5K7AVCNFSM6AAAAABLTZEFISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJZGQ3DKNBRGE>
. You are receiving this because you were mentioned.Message ID: @.*>
嗯,可以的,我们可以一起来搞,我也参与来一起看看。我更新了一下 feat/lpinpool 这个,现在把 feat/lpinpool
这个设置为保护分支了,后面我们都单独提 PR 到这个分支吧。我先写写 mint 方法,回头提个 PR 你可以帮忙 review,你有空可以写其他。
—
Reply to this email directly, view it on GitHub
<#104 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYSDDWXTF53UB4ZSRZ56PEDZPHEJPAVCNFSM6AAAAABLTZEFISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRSGA4TMMJVGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
现在分支上还没啥东西,不过我这两天正在整理,整体的设计会再细化一下,之前有些细节还不够明确 |
这个 PR 先关了,后面就建议不要提大的 PR 了,化整为零,一个个小方法的方式来提,这样也更好 review 和修改。 |
好的,收到
愚指导 ***@***.***> 于2024年8月8日周四 10:00写道:
… 这个 PR 先关了,后面就建议不要提大的 PR 了,化整为零,一个个小方法的方式来提,这样也更好 review 和修改。
—
Reply to this email directly, view it on GitHub
<#104 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYSDDWWWAXGUIOSMFZE5NUDZQLGNHAVCNFSM6AAAAABLTZEFISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZUG44TKNBSGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
No description provided.