Skip to content

Conversation

@rajesh-nodle
Copy link
Contributor

Part of #173

HighLights

  • Storage VestingSchedules is moved to bounded vector.
  • Upperlimit for VestingSchedules is Configured via MaxSchedule.
  • Introduce error code MaxScheduleOverflow.
  • UnitTest Extension :: Done
  • Runtime Integration :: Done
  • Storage Migration (try_runtime) :: Done
  • Benchmarking

TODO

Ready for Intake Review

Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
@codecov
Copy link

codecov bot commented Jul 12, 2022

Codecov Report

Merging #635 (7df01a9) into master (a2e0288) will increase coverage by 0.32%.
The diff coverage is 85.24%.

@@            Coverage Diff             @@
##           master     #635      +/-   ##
==========================================
+ Coverage   89.41%   89.74%   +0.32%     
==========================================
  Files          34       33       -1     
  Lines        6234     6238       +4     
==========================================
+ Hits         5574     5598      +24     
+ Misses        660      640      -20     
Impacted Files Coverage Δ
node/src/chain_spec.rs 0.00% <ø> (ø)
pallets/staking/src/benchmarking.rs 95.59% <ø> (ø)
runtimes/eden/src/pallets_nodle.rs 0.00% <ø> (ø)
runtimes/eden/src/pallets_util.rs 0.00% <ø> (ø)
pallets/allocations/src/lib.rs 74.54% <16.66%> (-8.15%) ⬇️
pallets/grants/src/lib.rs 62.96% <38.23%> (-1.33%) ⬇️
pallets/allocations/src/tests.rs 96.58% <60.00%> (-3.42%) ⬇️
pallets/grants/src/tests.rs 99.20% <99.44%> (-0.80%) ⬇️
pallets/allocations/src/benchmarking.rs 95.00% <100.00%> (+0.26%) ⬆️
pallets/grants/src/benchmarking.rs 100.00% <100.00%> (ø)
... and 4 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 a2e0288...7df01a9. Read the comment docs.

Copy link
Contributor

@simonsso simonsso left a comment

Choose a reason for hiding this comment

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

The bot is complaining about new code is not covered by tests, is there something we could do in this regard?

simonsso and others added 11 commits July 13, 2022 13:58
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
simonsso and others added 13 commits July 14, 2022 07:22
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
rajesh-nodle and others added 3 commits July 15, 2022 14:04
Co-authored-by: Alex Sedighi <alex@nodle.co>
Use the latest release of paritytech/srtool
* remove without_storage_info

Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>

* Update pallets/allocations/src/lib.rs

Co-authored-by: Fredrik Simonsson <simson@thesimson.net>

* Update pallets/allocations/src/lib.rs

Co-authored-by: Fredrik Simonsson <simson@thesimson.net>

* Update pallets/allocations/src/lib.rs

Co-authored-by: Fredrik Simonsson <simson@thesimson.net>

* Fix to CI/CD build try-runtime error

Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>

* Fix to CI/CD build try-runtime error (#637)

* chore: prepare for releasing 2.0.21 (#638)

* remove without_storage_info

Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>

* Update pallets/allocations/src/lib.rs

Co-authored-by: Fredrik Simonsson <simson@thesimson.net>

* Update pallets/allocations/src/lib.rs

Co-authored-by: Fredrik Simonsson <simson@thesimson.net>

* Update pallets/allocations/src/lib.rs

Co-authored-by: Fredrik Simonsson <simson@thesimson.net>

* Update pallets/allocations/src/lib.rs

Co-authored-by: Alex Sedighi <alex@nodle.co>

* Update pallets/allocations/src/lib.rs

Co-authored-by: Alex Sedighi <alex@nodle.co>

* intake review suggestions

Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>

Co-authored-by: R.Rajeshkumar <rajesh@nodle.com>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Alex Sedighi <alex@nodle.co>
R.Rajeshkumar and others added 19 commits July 19, 2022 22:11
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Co-authored-by: Fredrik Simonsson <simson@thesimson.net>
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
Signed-off-by: R.Rajeshkumar <rajesh@nodle.com>
@rajesh-nodle
Copy link
Contributor Author

rajesh-nodle commented Jul 20, 2022

Replaced by PR #642

@aliXsed aliXsed deleted the shamb0/v2_bounded_grants branch September 4, 2023 05:05
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