Skip to content
This repository has been archived by the owner on Mar 8, 2022. It is now read-only.

added "wallet/tokens/:id/sendtoaddress" #102

Merged
merged 1 commit into from
Jul 26, 2021
Merged

Conversation

fuxingloh
Copy link
Contributor

What kind of PR is this?:

/kind feature

What this PR does / why we need it:

Added "wallet/tokens/0/sendtoaddress" for future adaption of wallet/tokens/:id/sendtoaddress. This new endpoint will wait for automated block confirmation for the created transaction.

@codeclimate
Copy link

codeclimate bot commented Jul 24, 2021

Code Climate has analyzed commit 064fb46 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

View more on Code Climate.

@codecov
Copy link

codecov bot commented Jul 24, 2021

Codecov Report

Merging #102 (064fb46) into main (6731a1d) will increase coverage by 0.55%.
The diff coverage is 95.83%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #102      +/-   ##
==========================================
+ Coverage   78.10%   78.65%   +0.55%     
==========================================
  Files          34       34              
  Lines         548      567      +19     
  Branches       55       57       +2     
==========================================
+ Hits          428      446      +18     
- Misses        120      121       +1     
Impacted Files Coverage Δ
src/module.api/wallet.controller.ts 97.77% <95.83%> (-2.23%) ⬇️

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 6731a1d...064fb46. Read the comment docs.

@fuxingloh fuxingloh marked this pull request as ready for review July 24, 2021 14:35
@fuxingloh fuxingloh merged commit b36b5fc into main Jul 26, 2021
@fuxingloh fuxingloh deleted the fuxingloh/sendtoaddress branch July 26, 2021 06:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants