Skip to content

Commit

Permalink
Merge pull request #181 from 1Hive/158-check-failed-schedule-tx-with-…
Browse files Browse the repository at this point in the history
…bad-delay

Solve #158 
- Increased delay from 1min to 15min
- Fix minor typo in api subgraph link
  • Loading branch information
kamikazebr committed Mar 24, 2022
2 parents 154f144 + 394982d commit cc80485
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion packages/react-app/src/networks.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ export const networks = {
defaultEthNode: 'https://rinkeby.eth.aragon.network/',
questsSubgraph: 'https://api.thegraph.com/subgraphs/name/corantin/quests-subgraph',
governSubgraph: 'https://api.thegraph.com/subgraphs/name/corantin/govern-1hive-rinkeby',
celesteSubgraph: 'https://iapi.thegraph.com/subgraphs/name/corantin/celest-1hive-rinkeby',
celesteSubgraph: 'https://api.thegraph.com/subgraphs/name/corantin/celest-1hive-rinkeby',
questFactoryAddress: HardhatDeployement[4].rinkeby.contracts.QuestFactory.address,
governAddress: '0xa0F5e6759d49063040eAB18c1B0E684C45a4B4cA',
governQueueAddress: '0x19B918802eA9C71c500Ca481917F383f3992cDB0',
Expand Down
4 changes: 2 additions & 2 deletions packages/react-app/src/services/quest.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -177,8 +177,8 @@ async function generateScheduleContainer(
const lastBlockTimestamp = await getLastBlockTimestamp();

// A bit more than the execution delay
const executionTime = lastBlockTimestamp + erc3000Config.executionDelay + (extraDelaySec || 60); // Add 1 minute by default

const executionTime =
lastBlockTimestamp + erc3000Config.executionDelay + (extraDelaySec || 60 * 15); // Add 15 minutes by default
const evidenceIpfsHash = await pushObjectToIpfs(claimData.evidence);
const claimCall = encodeClaimAction(claimData, evidenceIpfsHash);

Expand Down

1 comment on commit cc80485

@vercel
Copy link

@vercel vercel bot commented on cc80485 Mar 24, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Successfully deployed to the following URLs:

Please sign in to comment.