fix: transfer payments contracts ownership to governor in migrate script

Signed-off-by: Tomás Migone <[email protected]>

Author: tmigone

packages/horizon/README.md

@@ -29,20 +29,20 @@ yarn build
 
 ## Deploy
 
+Note that this instructions will help you deploy Graph Horizon contracts, but no data service will be deployed. If you want to deploy the Subgraph Service please refer to the [Subgraph Service README](../subgraph-service/README.md) for deploy instructions.
+
 ### New deployment
 To deploy Graph Horizon from scratch run the following command:
 
 ```bash
 npx hardhat run scripts/deploy.ts --network hardhat
 ```
 
-Note that this will deploy a standalone version of Graph Horizon contracts, meaning the Subgraph Service or any other data service will not be deployed. If you want to deploy the Subgraph Service please refer to the [Subgraph Service README](../subgraph-service/README.md) for deploy instructions.
-
 ### Upgrade deployment
 To upgrade an existing deployment of the original Graph Protocol to Graph Horizon, run the following command:
 
 ```bash
-npx hardhat run scripts/migrate.ts --network hardhat
+npx hardhat run scripts/migrate.ts --network localhost
 ```
 
-Note that this will deploy a standalone version of Graph Horizon contracts, meaning the Subgraph Service or any other data service will not be deployed. If you want to deploy the Subgraph Service please refer to the [Subgraph Service README](../subgraph-service/README.md) for deploy instructions.
+Usually you would run this against a network (or a fork) where the original Graph Protocol was previously deployed.

packages/horizon/ignition/modules/core/GraphPayments.ts

@@ -40,6 +40,7 @@ export const MigrateGraphPaymentsModule = buildModule('GraphPayments', (m) => {
   const { GraphPaymentsProxyAdmin, GraphPaymentsProxy } = m.useModule(MigrateHorizonProxiesModule)
   const { Controller } = m.useModule(MigratePeripheryModule)
 
+  const governor = m.getAccount(1)
   const protocolPaymentCut = m.getParameter('protocolPaymentCut')
 
   // Deploy GraphPayments implementation
@@ -59,5 +60,7 @@ export const MigrateGraphPaymentsModule = buildModule('GraphPayments', (m) => {
       initArgs: [],
     })
 
+  m.call(GraphPaymentsProxyAdmin, 'transferOwnership', [governor], { after: [GraphPayments] })
+
   return { GraphPayments, GraphPaymentsProxyAdmin }
 })

packages/horizon/ignition/modules/core/PaymentsEscrow.ts

@@ -40,6 +40,7 @@ export const MigratePaymentsEscrowModule = buildModule('PaymentsEscrow', (m) =>
   const { PaymentsEscrowProxyAdmin, PaymentsEscrowProxy } = m.useModule(MigrateHorizonProxiesModule)
   const { Controller } = m.useModule(MigratePeripheryModule)
 
+  const governor = m.getAccount(1)
   const withdrawEscrowThawingPeriod = m.getParameter('withdrawEscrowThawingPeriod')
 
   // Deploy PaymentsEscrow implementation
@@ -59,5 +60,7 @@ export const MigratePaymentsEscrowModule = buildModule('PaymentsEscrow', (m) =>
       initArgs: [],
     })
 
+  m.call(PaymentsEscrowProxyAdmin, 'transferOwnership', [governor], { after: [PaymentsEscrow] })
+
   return { PaymentsEscrow, PaymentsEscrowProxyAdmin }
 })

packages/horizon/ignition/modules/migrate.ts

@@ -13,6 +13,7 @@ export default buildModule('GraphHorizon_Migrate', (m) => {
     GraphToken,
     GraphTokenGateway,
   } = m.useModule(MigratePeripheryModule)
+
   const {
     HorizonStaking,
     GraphPayments,