From 2238ccf40e98d4abd46377dd293ad6b0a0e27c5e Mon Sep 17 00:00:00 2001
From: Sebastian Kippe <sebastian@kip.pe>
Date: Thu, 14 Jan 2021 15:36:55 +0100
Subject: [PATCH] Fix confirmations being 1 block too late

block.number refers to the last mined block, so we need to add +1 to
wait from the current block on (the one in which this function is
executed).
---
 apps/contribution/contracts/Contribution.sol | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/apps/contribution/contracts/Contribution.sol b/apps/contribution/contracts/Contribution.sol
index 89bdfc7..e17f160 100644
--- a/apps/contribution/contracts/Contribution.sol
+++ b/apps/contribution/contracts/Contribution.sol
@@ -164,7 +164,7 @@ contract Contribution is AragonApp {
     if (contributionId < 10) {
       c.confirmedAtBlock = block.number;
     } else {
-      c.confirmedAtBlock = block.number + blocksToWait;
+      c.confirmedAtBlock = block.number + 1 + blocksToWait;
     }
 
     contributionsCount++;