Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The beneficiary parameter of claimRefund is replaced with refundee #1481

Merged
merged 31 commits into from
Nov 1, 2018
Merged
Changes from all commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
f2e5ed1
signing prefix added
Aniket-Engg Mar 10, 2018
ebdae17
Merge branch 'master' into master
Aniket-Engg Mar 13, 2018
2bf341c
Minor improvement
Aniket-Engg Mar 13, 2018
27c614a
Merge branch 'master' of https://github.com/Aniket-Engg/zeppelin-soli…
Aniket-Engg Mar 13, 2018
bc7ea7a
Tests changed
Aniket-Engg Mar 13, 2018
5a72021
Successfully tested
Aniket-Engg Mar 13, 2018
de0d2d4
Minor improvements
Aniket-Engg Mar 13, 2018
7f32e5c
Minor improvements
Aniket-Engg Mar 13, 2018
c3fb8a8
minor changes
Aniket-Engg Oct 1, 2018
68de840
Revert "Dangling commas are now required. (#1359)"
Aniket-Engg Oct 1, 2018
92b725f
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 1, 2018
1fe2cdf
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 1, 2018
75bc310
updates
Aniket-Engg Oct 1, 2018
9d9a5a0
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 3, 2018
10b1898
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 4, 2018
3858a0c
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 4, 2018
9484daa
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 5, 2018
5139019
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 8, 2018
3097a48
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 9, 2018
a46e176
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 10, 2018
fc9d1a7
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 11, 2018
23df893
fixes #1404
Aniket-Engg Oct 11, 2018
d8dd5ec
approve failing test
Aniket-Engg Oct 11, 2018
ae96caa
suggested changes done
Aniket-Engg Oct 16, 2018
e2fb20a
ISafeERC20 removed
Aniket-Engg Oct 16, 2018
52c7870
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 16, 2018
998dfd7
Merge branch 'fix/#1404' of https://github.com/Aniket-Engg/zeppelin-s…
Aniket-Engg Oct 16, 2018
c8963b1
Merge remote-tracking branch 'upstream/master'
Aniket-Engg Oct 17, 2018
7ffe179
conflicts
Aniket-Engg Oct 22, 2018
5f88845
conflict fixes
Aniket-Engg Oct 22, 2018
b0848c9
fixes #1456
Aniket-Engg Oct 24, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions contracts/crowdsale/distribution/RefundableCrowdsale.sol
Original file line number Diff line number Diff line change
Expand Up @@ -50,13 +50,13 @@ contract RefundableCrowdsale is FinalizableCrowdsale {

/**
* @dev Investors can claim refunds here if crowdsale is unsuccessful
* @param beneficiary Whose refund will be claimed.
* @param refundee Whose refund will be claimed.
*/
function claimRefund(address beneficiary) public {
function claimRefund(address refundee) public {
require(finalized());
require(!goalReached());

_escrow.withdraw(beneficiary);
_escrow.withdraw(refundee);
}

/**
Expand Down