firoorg / ztm

A REST service running on top of Zcoin Exodus to provides rich and easy to use APIs for managing the token
https://ztm.readthedocs.io
MIT License
4 stars 3 forks source link

Token granting #157

Closed thebevrishot closed 4 years ago

thebevrishot commented 4 years ago

This PR is not include EndToEnd test.

closes https://github.com/zcoinofficial/ztm/issues/82 https://github.com/zcoinofficial/ztm/issues/154

codecov[bot] commented 4 years ago

Codecov Report

Merging #157 into master will increase coverage by 1.18%. The diff coverage is 94.95%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #157      +/-   ##
==========================================
+ Coverage   86.99%   88.18%   +1.18%     
==========================================
  Files         134      138       +4     
  Lines        4615     4682      +67     
==========================================
+ Hits         4015     4129     +114     
+ Misses        545      529      -16     
+ Partials       55       24      -31
Impacted Files Coverage Δ
src/Ztm.Zcoin.NBitcoin.Json/UInt256Converter.cs 100% <ø> (ø)
src/Ztm.WebApi/Models/IssueTokenRequest.cs 100% <ø> (ø)
src/Ztm.WebApi/Startup.cs 0% <0%> (ø) :arrow_up:
...tionConfirmation/TransactionConfirmationWatcher.cs 95.83% <100%> (-0.02%) :arrow_down:
src/Ztm.Configuration/ConfigurationExtensions.cs 100% <100%> (ø) :arrow_up:
src/Ztm.WebApi/AddressPools/RpcAddressGenerator.cs 100% <100%> (ø) :arrow_up:
...Ztm.Zcoin.NBitcoin.Json/BitcoinAddressConverter.cs 100% <100%> (ø)
src/Ztm.Zcoin.Rpc/WalletRpc.cs 100% <100%> (ø)
src/Ztm.Zcoin.Rpc/ExodusTransactionInformation.cs 100% <100%> (ø) :arrow_up:
src/Ztm.Zcoin.Rpc/ExodusInformationRpc.cs 100% <100%> (ø)
... and 18 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 0c6a1ae...af2a86c. Read the comment docs.

lgtm-com[bot] commented 4 years ago

This pull request introduces 8 alerts when merging 81ad6545ce3e6a04aa04faa19d0e87cb1ae37173 into 6fef6e3f5d0df50b2fe4f12aa1284e92d1f0d673 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 4 years ago

This pull request introduces 1 alert when merging 241207bbe18085c60f3db7a86fe1cf3cd4195f10 into 0ea6ead6cba60648f927cc3b477e675be93523b3 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 4 years ago

This pull request introduces 1 alert when merging 543f686af2bd8ee2f2234bfda83a59a275f2a51f into 0ea6ead6cba60648f927cc3b477e675be93523b3 - view on LGTM.com

new alerts: