documentation icon indicating copy to clipboard operation
documentation copied to clipboard

Added native token top-up and withdraw functionality in VRFv2PlusSubscriptionManager contract.

Open SyedAsadKazmi opened this issue 1 year ago • 4 comments

Some devs are asking for this functionality, as can be seen from this Eth-StackExchange post, so I've added thetopUpSubscriptionWithNativeToken and withdrawNativeToken functions to support the same.

SyedAsadKazmi avatar Oct 14 '24 14:10 SyedAsadKazmi

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview Dec 19, 2024 3:38pm

vercel[bot] avatar Oct 14 '24 14:10 vercel[bot]

Hi @SyedAsadKazmi , thanks for the contribution. Since this PR is to help users to fund the vrf 2.5 subscription with native tokens, I think it might be a good idea that you can also add a new function to allow them to request randomness with native payment.

For now, although users can fund the subscription with native tokens, there is no function in the contract for them to request randomness with native balance.

QingyangKong avatar Dec 11 '24 06:12 QingyangKong

Someone is attempting to deploy a commit to the Chainlink Labs Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Dec 13 '24 01:12 vercel[bot]

Hi @SyedAsadKazmi , thanks for the contribution. Since this PR is to help users to fund the vrf 2.5 subscription with native tokens, I think it might be a good idea that you can also add a new function to allow them to request randomness with native payment.

For now, although users can fund the subscription with native tokens, there is no function in the contract for them to request randomness with native balance.

Hello @QingyangKong,

Thanks for the feedback. I've added the enableNativePayment argument to the requestRandomWords() function. Please check.

SyedAsadKazmi avatar Dec 13 '24 01:12 SyedAsadKazmi