Skip to content

Quick fix to swagger docs#3933

Merged
melekes merged 1 commit intomasterfrom
marko/guickfixswagger
Sep 2, 2019
Merged

Quick fix to swagger docs#3933
melekes merged 1 commit intomasterfrom
marko/guickfixswagger

Conversation

@tac0turtle
Copy link
Contributor

  • switch the data that was represented in /unconfirmed_txs and num_unconfirmed_txs

Signed-off-by: Marko Baricevic marbar3778@yahoo.com

  • Referenced an issue explaining the need for the change
  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

- switch the data that was represented in `/unconfirmed_txs` and `num_unconfirmed_txs`

Signed-off-by: Marko Baricevic <marbar3778@yahoo.com>
@tac0turtle tac0turtle added the C:docs Component: Documentation label Sep 2, 2019
@tac0turtle tac0turtle requested a review from melekes September 2, 2019 17:50
@codecov-io
Copy link

Codecov Report

Merging #3933 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3933      +/-   ##
==========================================
- Coverage      67%   66.97%   -0.04%     
==========================================
  Files         219      219              
  Lines       18207    18199       -8     
==========================================
- Hits        12200    12189      -11     
- Misses       5125     5126       +1     
- Partials      882      884       +2
Impacted Files Coverage Δ
rpc/core/mempool.go 0% <ø> (ø) ⬆️
p2p/pex/pex_reactor.go 82.6% <0%> (-0.58%) ⬇️
consensus/reactor.go 79.13% <0%> (-0.36%) ⬇️
rpc/client/helpers.go 87.5% <0%> (ø) ⬆️
blockchain/v0/pool.go 80.98% <0%> (+0.32%) ⬆️

@melekes melekes merged commit 72785a2 into master Sep 2, 2019
@melekes melekes deleted the marko/guickfixswagger branch September 2, 2019 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C:docs Component: Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants