feat(contracts): create subscription in constructor

zmalatrax · unknownunknown1 · commit eb49a063b192 · 2023-10-29T01:17:26.000+10:00
diff --git a/contracts/src/rng/VRFSubscriptionManagerV2.sol b/contracts/src/rng/VRFSubscriptionManagerV2.sol
@@ -72,6 +72,7 @@ contract VRFSubscriptionManagerV2 {
         vrfCoordinator = VRFCoordinatorV2Interface(_vrfCoordinator);
         linkToken = LinkTokenInterface(_linkToken);
         governor = _governor;
+        createNewSubscription();
     }
 
     // ************************************* //
@@ -112,7 +113,7 @@ contract VRFSubscriptionManagerV2 {
     /**
      * @dev Creates a new subscription, overriding the previous one to be manageable by the contract.
      */
-    function createNewSubscription() external onlyByGovernor {
+    function createNewSubscription() public onlyByGovernor {
         subscriptionId = vrfCoordinator.createSubscription();
     }
 

Original file line number	Diff line number	Diff line change
`@@ -72,6 +72,7 @@ contract VRFSubscriptionManagerV2 {`
`72`	`72`	`vrfCoordinator = VRFCoordinatorV2Interface(_vrfCoordinator);`
`73`	`73`	`linkToken = LinkTokenInterface(_linkToken);`
`74`	`74`	`governor = _governor;`
	`75`	`+ createNewSubscription();`
`75`	`76`	`}`
`76`	`77`
`77`	`78`	`// ************************************* //`
`@@ -112,7 +113,7 @@ contract VRFSubscriptionManagerV2 {`
`112`	`113`	`/**`
`113`	`114`	`* @dev Creates a new subscription, overriding the previous one to be manageable by the contract.`
`114`	`115`	`*/`
`115`		`- function createNewSubscription() external onlyByGovernor {`
	`116`	`+ function createNewSubscription() public onlyByGovernor {`
`116`	`117`	`subscriptionId = vrfCoordinator.createSubscription();`
`117`	`118`	`}`
`118`	`119`