Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend adapter and decoders #43

Merged
merged 2 commits into from
Feb 11, 2020
Merged

Extend adapter and decoders #43

merged 2 commits into from
Feb 11, 2020

Conversation

joe94
Copy link
Member

@joe94 joe94 commented Feb 10, 2020

-Extend wrphttp adapter to include original payload (useful for happy path for users who correctly fill out their WRPs)
-Wrap existing functions to fetch wrp entity from either headers or body

@codecov-io
Copy link

codecov-io commented Feb 10, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@6e537b4). Click here to learn what that means.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #43   +/-   ##
=========================================
  Coverage          ?   47.23%           
=========================================
  Files             ?       15           
  Lines             ?     3102           
  Branches          ?        0           
=========================================
  Hits              ?     1465           
  Misses            ?     1478           
  Partials          ?      159
Impacted Files Coverage Δ
wrphttp/decoders.go 74.07% <0%> (ø)

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 6e537b4...1afa0a7. Read the comment docs.

@joe94
Copy link
Member Author

joe94 commented Feb 11, 2020

fixes #37

@joe94 joe94 merged commit 4ca6fd2 into master Feb 11, 2020
@joe94 joe94 deleted the feature/wrpHandler branch February 11, 2020 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants