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

Chainlink VRF V2 升级至 VRF V2.5 #816

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

Conversation

eddiehsu66
Copy link

@eddiehsu66 eddiehsu66 commented Oct 17, 2024

What type of PR is this (这是什么类型的PR)

  • Typo(勘误)
  • BUG(程序错误)
  • Improvement(提升)
  • Feature(新特性)

Which issue(s) this PR fixes(Optional) (这个PR 修复了什么问题 (可选择))

What this PR does / why we need it (这个PR 做了什么/ 我们为什么需要这个PR)

当前Chainlink官网不支持VRF V2,获取的subId从uint64变为了uint256,需要进行迁移

@XdpCs
Copy link
Collaborator

XdpCs commented Nov 2, 2024

@eddiehsu66 pr需要引用解决的相关问题 我已经帮你引用上了 下次希望能够引用一下解决的问题 你的代码这几天 我会去本地测试的 谢谢你的贡献

@XdpCs
Copy link
Collaborator

XdpCs commented Nov 4, 2024

#607 是需要支持预言机本地化 如果可以的话 能否在这个pr中支持

@eddiehsu66
Copy link
Author

okok,我重新看一下文档整理一下

#607 是需要支持预言机本地化 如果可以的话 能否在这个pr中支持

@XdpCs
Copy link
Collaborator

XdpCs commented Nov 7, 2024

okok,我重新看一下文档整理一下

#607 是需要支持预言机本地化 如果可以的话 能否在这个pr中支持

好 辛苦大佬

@eddiehsu66
Copy link
Author

#607 是需要支持预言机本地化 如果可以的话 能否在这个pr中支持

OKOK,已支持预言机本地化测试,已提交

@XdpCs
Copy link
Collaborator

XdpCs commented Nov 10, 2024

#607 是需要支持预言机本地化 如果可以的话 能否在这个pr中支持

OKOK,已支持预言机本地化测试,已提交

辛苦辛苦 争取下周 review完

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.

(39. 链上随机数) 适配于Chainlink VRF v2.5的Random.sol
2 participants