PolymeshAssociation / Polymesh

Node for Polymesh Blockchain
https://polymesh.network/
GNU General Public License v3.0
165 stars 47 forks source link

Fix upgradable polymesh api and tracker. #1478

Closed Neopallium closed 1 year ago

Neopallium commented 1 year ago

There is an issue with payable contract methods making DelegateCalls to the upgradable Polymesh Ink! api. A work around is to make the API methods also payable.

Also the upgrade tracker contract was alway using the oldest version, not the newest.

codecov[bot] commented 1 year ago

Codecov Report

Merging #1478 (7f82c3d) into develop (de418e9) will increase coverage by 0.00%. The diff coverage is 0.00%.

@@           Coverage Diff            @@
##           develop    #1478   +/-   ##
========================================
  Coverage    53.94%   53.94%           
========================================
  Files          151      151           
  Lines        32890    32890           
========================================
+ Hits         17741    17742    +1     
+ Misses       15149    15148    -1     
Impacted Files Coverage Δ
contracts/upgradeable-polymesh-ink/src/macros.rs 44.33% <0.00%> (ø)

... and 6 files with indirect coverage changes