Fix wrong condition, resulting in 404 for valid NodeInfo endpoints #464

Merged
elegaanz merged 2 commits from fix-nodeinfo into master 2019-03-12 16:00:23 +00:00
elegaanz commented 2019-03-12 13:32:17 +00:00 (Migrated from github.com)

Once we have test for the main crate, this should definitely be tested.

Once we have test for the main crate, this should definitely be tested.
codecov[bot] commented 2019-03-12 14:28:31 +00:00 (Migrated from github.com)

Codecov Report

Merging #464 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #464   +/-   ##
=======================================
  Coverage   27.23%   27.23%           
=======================================
  Files          64       64           
  Lines        7310     7310           
=======================================
  Hits         1991     1991           
  Misses       5319     5319
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/464?src=pr&el=h1) Report > Merging [#464](https://codecov.io/gh/Plume-org/Plume/pull/464?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/fe6e69d7c4408a12733d2150309c92811691ee12?src=pr&el=desc) will **not change** coverage. > The diff coverage is `0%`. ```diff @@ Coverage Diff @@ ## master #464 +/- ## ======================================= Coverage 27.23% 27.23% ======================================= Files 64 64 Lines 7310 7310 ======================================= Hits 1991 1991 Misses 5319 5319 ```
trinity-1686a approved these changes 2019-03-12 15:46:59 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#464
No description provided.