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

fetch reducer #428

Merged
merged 10 commits into from
Sep 23, 2019
Merged

fetch reducer #428

merged 10 commits into from
Sep 23, 2019

Conversation

acatl
Copy link
Collaborator

@acatl acatl commented Sep 11, 2019

What:

Adds a reducer to fetch resources, this reducer is just a wrapper for the node-fetch package.

closes #426

Checklist:

  • Has Breaking changes
  • Documentation
  • Tests
  • Ready to be merged
  • Added username to all-contributors list

@acatl acatl added this to the v6 Feature Complete milestone Sep 11, 2019
@acatl acatl self-assigned this Sep 11, 2019

For documentation on this module's api: [http://data-point.github.com/reducers/fetch](http://data-point.github.com/reducers/fetch).

For data-point's documentation visit its [home page](http://data-point.github.com) ([http://data-point.github.com](http://data-point.github.com)).
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this and the url from above are wrong, will update them once we know their final destination.

@codecov
Copy link

codecov bot commented Sep 11, 2019

Codecov Report

Merging #428 into canary will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           canary   #428   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          29     35    +6     
  Lines         482    530   +48     
  Branches       74     84   +10     
=====================================
+ Hits          482    530   +48
Impacted Files Coverage Δ
packages/data-point-fetch/Fetch.js 100% <100%> (ø)
packages/data-point/Reducer.js 100% <100%> (ø)
packages/data-point/createReducer.js 100% <100%> (ø)
packages/data-point/ReducerEntity.js 100% <100%> (ø)
packages/data-point/index.js 100% <0%> (ø)
packages/data-point/src/index.js 100% <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 e7b0b3f...3d462f9. Read the comment docs.

@acatl acatl requested a review from csarnataro September 13, 2019 15:41
csarnataro
csarnataro previously approved these changes Sep 13, 2019
@acatl acatl merged commit 28034de into ViacomInc:canary Sep 23, 2019
@acatl acatl deleted the 426-fetch-reducer branch September 23, 2019 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants