fix: add recieve function

contracts/strategies/_poc/qv-hackathon/SchemaResolver.sol

@@ -7,6 +7,7 @@ pragma solidity 0.8.19;
 //  - updated import paths
 //  - updated _eas to make it mutable
 //  - replaced constructor  __SchemaResolver_init()
+//  - commented out receive() function
 // Reason for change:
 //  - wanted it to set when HackathonQVStrategy.initialize() is invoked
 import {IEAS, Attestation} from "eas-contracts/IEAS.sol";
@@ -59,11 +60,12 @@ abstract contract SchemaResolver is ISchemaResolver {
     /**
      * @dev ETH callback.
      */
-    receive() external payable virtual {
-        if (!isPayable()) {
-            revert NotPayable();
-        }
-    }
+    // TODO: Revisit this logic
+    // receive() external payable virtual {
+    //     if (!isPayable()) {
+    //         revert NotPayable();
+    //     }
+    // }
 
     /**
      * @inheritdoc ISchemaResolver

contracts/strategies/donation-voting-merkle-base/DonationVotingMerkleDistributionBaseStrategy.sol

@@ -847,6 +847,6 @@ abstract contract DonationVotingMerkleDistributionBaseStrategy is Native, BaseSt
         return (rowIndex, colIndex, statusesBitMap[rowIndex]);
     }
 
-    /// @notice Contract should be able to receive ETH
+    /// @notice Contract should be able to receive NATIVE
     receive() external payable {}
 }

contracts/strategies/qv-base/QVBaseStrategy.sol

@@ -616,4 +616,7 @@ abstract contract QVBaseStrategy is BaseStrategy {
             revert INVALID();
         }
     }
+
+    /// @notice Contract should be able to receive NATIVE
+    receive() external payable {}
 }

test/foundry/strategies/HackathonQVStrategy.t.sol

@@ -63,8 +63,8 @@ contract HackathonQVStrategyTest is QVBaseStrategyTest, Native {
         __setAllowedRecipientId();
     }
 
-    function _createStrategy() internal override returns (address) {
-        return address(new HackathonQVStrategy(address(allo()), "MockStrategy"));
+    function _createStrategy() internal override returns (address payable) {
+        return payable(address(new HackathonQVStrategy(address(allo()), "MockStrategy")));
     }
 
     function hQvStrategy() internal view returns (HackathonQVStrategy) {

test/foundry/strategies/QVBaseStrategy.t.sol

@@ -49,7 +49,7 @@ contract QVBaseStrategyTest is Test, AlloSetup, RegistrySetupFull, StrategySetup
     uint64 public allocationStartTime;
     uint64 public allocationEndTime;
 
-    address internal _strategy;
+    address payable internal _strategy;
     MockERC20 public token;
     Metadata public poolMetadata;
 
@@ -93,8 +93,8 @@ contract QVBaseStrategyTest is Test, AlloSetup, RegistrySetupFull, StrategySetup
         _initialize();
     }
 
-    function _createStrategy() internal virtual returns (address) {
-        return address(new QVBaseStrategyTestMock(address(allo()), "MockStrategy"));
+    function _createStrategy() internal virtual returns (address payable) {
+        return payable(address(new QVBaseStrategyTestMock(address(allo()), "MockStrategy")));
     }
 
     function _initialize() internal virtual {
@@ -139,7 +139,7 @@ contract QVBaseStrategyTest is Test, AlloSetup, RegistrySetupFull, StrategySetup
 
     function test_initialize_UNAUTHORIZED() public virtual {
         vm.startPrank(allo_owner());
-        address strategy = address(new QVBaseStrategyTestMock(address(allo()), "MockStrategy"));
+        address payable strategy = payable(address(new QVBaseStrategyTestMock(address(allo()), "MockStrategy")));
         vm.expectRevert(UNAUTHORIZED.selector);
         vm.stopPrank();
         vm.startPrank(randomAddress());

test/foundry/strategies/QVGovernanceERC20Votes.t.sol

@@ -26,8 +26,8 @@ contract QVGovernanceERC20VotesTest is QVBaseStrategyTest {
         super.setUp();
     }
 
-    function _createStrategy() internal override returns (address) {
-        return address(new QVGovernanceERC20Votes(address(allo()), "MockStrategy"));
+    function _createStrategy() internal override returns (address payable) {
+        return payable(address(new QVGovernanceERC20Votes(address(allo()), "MockStrategy")));
     }
 
     function qvGovStrategy() internal view returns (QVGovernanceERC20Votes) {

test/foundry/strategies/QVNftTieredStrategy.t.sol

@@ -29,8 +29,8 @@ contract QVNftTieredStrategyTest is QVBaseStrategyTest {
         super.setUp();
     }
 
-    function _createStrategy() internal override returns (address) {
-        return address(new QVNftTieredStrategy(address(allo()), "MockStrategy"));
+    function _createStrategy() internal override returns (address payable) {
+        return payable(address(new QVNftTieredStrategy(address(allo()), "MockStrategy")));
     }
 
     function qvNftStrategy() internal view returns (QVNftTieredStrategy) {

test/foundry/strategies/QVSimpleStrategy.t.sol

@@ -21,8 +21,8 @@ contract QVSimpleStrategyTest is QVBaseStrategyTest {
         super.setUp();
     }
 
-    function _createStrategy() internal override returns (address) {
-        return address(new QVSimpleStrategy(address(allo()), "MockStrategy"));
+    function _createStrategy() internal override returns (address payable) {
+        return payable(address(new QVSimpleStrategy(address(allo()), "MockStrategy")));
     }
 
     function _initialize() internal override {