remove ownable from managerhelper. up to 2.0.0

contracts/ReleaseManager.sol

@@ -175,7 +175,6 @@ contract ReleaseManager is OwnableUpgradeable, IReleaseManager {
     function setWarnings(address factoryAddress, string memory url)  public onlyOwner {
         warnings[factoryAddress] = url;
     }
-
 
 
     // which adds to instances[address] which looks for msg.sender or _msgSender() is the factory address if it supports EIP2771 just in case. 
@@ -188,7 +187,7 @@ contract ReleaseManager is OwnableUpgradeable, IReleaseManager {
         return (instances[instanceAddress].factoryAddress == address(0) ? false : true);
     }
 
-    function checkFactory(address factoryAddress) external view returns(bool) {
+    function checkFactory(address factoryAddress) external view override returns(bool) {
         return (factories[factoryAddress].factoryIndex == 0 ? false : true);
     }
 

contracts/ReleaseManagerHelper.sol

@@ -1,35 +1,41 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.0;
 
-// import "@openzeppelin/contracts-upgradeable/access/OwnableUpgradeable.sol";
 import "./interfaces/IReleaseManager.sol";
-import "@openzeppelin/contracts/access/Ownable.sol";
 import "@openzeppelin/contracts/utils/Address.sol";
 
-contract ReleaseManagerHelper is Ownable{
+abstract contract ReleaseManagerHelper {
     using Address for address;
-    address public releaseManager;
+    address private _releaseManager;
 
-    error RegisterReleaseManagerFirst();
-    error ReleaseManagerIsNoneContract(address addr);
-    error ReleaseManagerAlreadySetup(address addr);
-
-    function registerInstance(address stateContractAddress) internal {
-        if (releaseManager == address(0)) {
-            revert RegisterReleaseManagerFirst();
+    error ReleaseManagerInvalid(address addr);
+
+    /**
+    * @notice need to register release manager when factory(CommunityCoinFactory, IncomeContractFactory, etc) deployed
+    */
+    constructor(address releaseManagerAddr) {
+        if (
+            releaseManagerAddr.isContract() == false ||
+            releaseManagerAddr == address(0)
+        ) {
+            revert ReleaseManagerInvalid(releaseManagerAddr);
         }
-        IReleaseManager(releaseManager).registerInstance(stateContractAddress);
+
+        _releaseManager = releaseManagerAddr;
     }
-
-    function registerReleaseManager(address releaseManager_) public onlyOwner {
-        if (releaseManager_.isContract() == false) {
-            revert ReleaseManagerIsNoneContract(releaseManager_);
-        }
-        if (releaseManager != address(0)) {
-            revert ReleaseManagerAlreadySetup(releaseManager_);
-        }
 
-        releaseManager = releaseManager_;
+    /**
+    * @notice view release manager address that was regsted when factory deployed
+    */
+    function releaseManager() public view returns(address) {
+        return _releaseManager;
     }
 
+    /**
+    * @param instanceAddress address that was produced by factory need to be registered in ReleaseManager. Usually such method should be called after produce/produceDeterministic
+    */
+    function registerInstance(address instanceAddress) internal {
+        IReleaseManager(_releaseManager).registerInstance(instanceAddress);
+    }
+
 }

contracts/mocks/examples/FactoryEx1.sol

@@ -19,9 +19,12 @@ contract FactoryEx1 is Ownable, CostManagerFactoryHelper, ReleaseManagerHelper {
     event InstanceCreated(address instance, uint256 instancesCount);
 
     constructor(
-        address impl
+        address impl,
+        address costManagerAddress,
+        address releaseManagerAddress
     )
-        CostManagerFactoryHelper(address(0))
+        CostManagerFactoryHelper(costManagerAddress)
+        ReleaseManagerHelper(releaseManagerAddress)
     {
         implementation = impl;
     }