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

Adding increaseAllowance and decreaseAllowance methods #1196

Merged
merged 5 commits into from
Oct 7, 2019

Conversation

mrsmkl
Copy link
Contributor

@mrsmkl mrsmkl commented Oct 3, 2019

Description

Added these methods to stable and gold token. Also added then to contractkit.

Tested

Will add tests soon.

Other changes

Related issues

Backwards compatibility

@mrsmkl mrsmkl requested review from asaj and m-chrzan as code owners October 3, 2019 08:41
@asaj asaj assigned mrsmkl and unassigned asaj Oct 4, 2019
@asaj asaj added the automerge Have PR merge automatically when checks pass label Oct 5, 2019
@codecov
Copy link

codecov bot commented Oct 7, 2019

Codecov Report

Merging #1196 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1196   +/-   ##
=======================================
  Coverage   66.47%   66.47%           
=======================================
  Files         262      262           
  Lines        7648     7648           
  Branches      509      509           
=======================================
  Hits         5084     5084           
  Misses       2462     2462           
  Partials      102      102
Flag Coverage Δ
#mobile 66.47% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0d164b0...7635769. Read the comment docs.

@ashishb ashishb merged commit 7ebaa10 into celo-org:master Oct 7, 2019
aaronmgdr added a commit that referenced this pull request Oct 8, 2019
* master: (35 commits)
  [Wallet] Fix top of emojis cut off in the activity feed (#1243)
  Adding a contract to store minimum required client version (#1081)
  Revert "Feature #909 proxy delegatecall (#1152)" (#1241)
  Use ContractKit to get addresses for Blockchain API (#1175)
  Feature #909 proxy delegatecall (#1152)
  Fix Faucet done message (#1217)
  Updated SETUP.md with new yarn process (#1224)
  Adding `increaseAllowance` and `decreaseAllowance` methods (#1196)
  extracting function signatures (#1061)
  Fix integration hardcode (#1208)
  Fixing flaky governance test (#1155)
  Restore CI branch (#1223)
  [wallet] e2e back to green (#1210)
  [Wallet] Implement new import wallet flow designs (#1209)
  [Wallet] Fix disable conditions for butons on Enter Invite screen (#1214)
  [protocol] Rename infrastructureFraction to proposerFraction (#1174)
  [ck] Proper promise treatment to avoid UnhandledPromises (#1219)
  [ck] Transform StableToken parameters from fixidity format (#1218)
  [wallet]Store encrypted local signing key (#1188)
  2019-10-03 alfajores deployment (#1200)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Have PR merge automatically when checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Users SBAT use increaseAllowance and decreaseAllowance in ERC20 contracts
3 participants