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

Feature/skale 4448 allow wallets store skl #653

Draft
wants to merge 47 commits into
base: feature/SKALE-3749-schain-pricing
Choose a base branch
from

Conversation

yavrsky
Copy link
Contributor

@yavrsky yavrsky commented Aug 17, 2021

Allow Wallets to store SKL

payvint added 30 commits July 22, 2021 12:45
payvint and others added 16 commits August 4, 2021 14:06
…t-private-keys

Enhancement/skale 2510 adopt private keys
…events

Enhancement/skale 3785 add events
Bumps [web3](https://github.com/ethereum/web3.js) from 1.4.0 to 1.5.1.
- [Release notes](https://github.com/ethereum/web3.js/releases)
- [Changelog](https://github.com/ChainSafe/web3.js/blob/1.x/CHANGELOG.md)
- [Commits](web3/web3.js@v1.4.0...v1.5.1)

---
updated-dependencies:
- dependency-name: web3
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [ethers](https://github.com/ethers-io/ethers.js/tree/HEAD/packages/ethers) from 5.4.2 to 5.4.4.
- [Release notes](https://github.com/ethers-io/ethers.js/releases)
- [Changelog](https://github.com/ethers-io/ethers.js/blob/master/CHANGELOG.md)
- [Commits](https://github.com/ethers-io/ethers.js/commits/v5.4.4/packages/ethers)

---
updated-dependencies:
- dependency-name: ethers
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
…ers-5.4.4

Bump ethers from 5.4.2 to 5.4.4
@yavrsky yavrsky requested a review from a team as a code owner August 17, 2021 14:19
@codecov
Copy link

codecov bot commented Aug 17, 2021

Codecov Report

Merging #653 (7b540ec) into feature/SKALE-3749-schain-pricing (3afebe3) will increase coverage by 0.00%.
The diff coverage is 98.50%.

Impacted file tree graph

@@                        Coverage Diff                         @@
##           feature/SKALE-3749-schain-pricing     #653   +/-   ##
==================================================================
  Coverage                              95.39%   95.40%           
==================================================================
  Files                                     38       38           
  Lines                                   2584     2611   +27     
  Branches                                 414      409    -5     
==================================================================
+ Hits                                    2465     2491   +26     
- Misses                                   119      120    +1     
Impacted Files Coverage Δ
contracts/NodeRotation.sol 88.23% <0.00%> (-1.32%) ⬇️
contracts/SkaleDKG.sol 99.39% <ø> (ø)
contracts/BountyV2.sol 89.20% <100.00%> (+0.15%) ⬆️
contracts/ConstantsHolder.sol 100.00% <100.00%> (ø)
contracts/Nodes.sol 95.11% <100.00%> (+0.09%) ⬆️
contracts/Schains.sol 100.00% <100.00%> (ø)
contracts/SchainsInternal.sol 92.80% <100.00%> (+0.05%) ⬆️
contracts/SkaleManager.sol 100.00% <100.00%> (ø)
contracts/SlashingTable.sol 100.00% <100.00%> (ø)
contracts/Wallets.sol 98.63% <100.00%> (+0.14%) ⬆️
... and 2 more

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 3afebe3...7b540ec. Read the comment docs.

// /**
// * @dev Returns the current price in SKL tokens for given Schain type and lifetime.
// */
// function getSchainPrice(uint typeOfSchain, uint lifetime) public view returns (uint) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Should it be removed?

msg.sender.transfer(amount);
}

function storeSchainDeposit(bytes32 schainHash, uint deposit) external allow("Schains") {
_schainDeposits[schainHash] += deposit;
Copy link
Contributor

Choose a reason for hiding this comment

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

SafeMath is missing.

msg.sender.transfer(amount);
}

function storeSchainDeposit(bytes32 schainHash, uint deposit) external allow("Schains") {
Copy link
Contributor

Choose a reason for hiding this comment

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

"IncreaseSchainDeposit"?

@DimaStebaev DimaStebaev marked this pull request as draft September 3, 2021 10:32
@yavrsky yavrsky force-pushed the feature/SKALE-4448-allow-wallets-store-skl branch from 7699f7b to 7b540ec Compare September 27, 2021 15:48
@skale-cla
Copy link

skale-cla commented Oct 19, 2021

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants