Skip to content

Commit

Permalink
fix: ProvisionAccepted now ProvisionPendingParametersAccepted.
Browse files Browse the repository at this point in the history
  • Loading branch information
MoonBoi9001 committed Sep 20, 2024
1 parent a62ba22 commit 2c2caab
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ interface IDataService {
* @notice Emitted when a service provider accepts a provision in {Graph Horizon staking contract}.
* @param serviceProvider The address of the service provider.
*/
event ProvisionAccepted(address indexed serviceProvider);
event ProvisionPendingParametersAccepted(address indexed serviceProvider);

/**
* @notice Emitted when a service provider starts providing the service.
Expand Down Expand Up @@ -88,7 +88,7 @@ interface IDataService {
* contract}.
* @dev Provides a way for the data service to validate and accept provision parameter changes. Call {_acceptProvision}.
*
* Emits a {ProvisionAccepted} event.
* Emits a {ProvisionPendingParametersAccepted} event.
*
* @param serviceProvider The address of the service provider.
* @param data Custom data, usage defined by the data service.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ abstract contract ProvisionManager is Initializable, GraphDirectory, ProvisionMa
* the {HorizonStaking} contract.
* @dev Checks the pending provision parameters, not the current ones.
*
* Emits a {ProvisionAccepted} event.
* Emits a {ProvisionPendingParametersAccepted} event.
*
* @param _serviceProvider The address of the service provider.
*/
Expand Down
4 changes: 2 additions & 2 deletions packages/subgraph-service/contracts/SubgraphService.sol
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ contract SubgraphService is
* - Must have previously staged provision parameters, using {IHorizonStaking-setProvisionParameters}
* - The new provision parameters must be valid according to the subgraph service rules
*
* Emits a {ProvisionAccepted} event
* Emits a {ProvisionPendingParametersAccepted} event
*
* @param indexer The address of the indexer to accept the provision for
*/
Expand All @@ -140,7 +140,7 @@ contract SubgraphService is
) external override onlyAuthorizedForProvision(indexer) onlyRegisteredIndexer(indexer) whenNotPaused {
_checkProvisionTokens(indexer);
_acceptProvisionParameters(indexer);
emit ProvisionAccepted(indexer);
emit ProvisionPendingParametersAccepted(indexer);
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ contract SubgraphServiceTest is SubgraphServiceSharedTest {
uint64 thawingPeriodPending = provision.thawingPeriodPending;

vm.expectEmit(address(subgraphService));
emit IDataService.ProvisionAccepted(_indexer);
emit IDataService.ProvisionPendingParametersAccepted(_indexer);

// Accept provision
subgraphService.acceptProvisionPendingParameters(_indexer, _data);
Expand Down

0 comments on commit 2c2caab

Please sign in to comment.