Skip to content

Conversation

@prescottprue
Copy link
Owner

@prescottprue prescottprue commented Dec 11, 2017

Description

  • feat(HOCs): Removed recompose's withContext from withFirebase and withFirestore in order to support react v16.*.* - bug(withFirebase): "Warning: Failed child context type..." when using react v16 #337
  • feat(HOCs): props.dispatch added to withFirebase
  • feat(core): expose reducer as reducer to match common redux panther also seen with other
    libraries such as redux-form
  • fix(storage): progress: true option on upload would cause dispatch of FILE_UPLOAD_START to occur twice
  • feat(tests): drastically simplified tests (mostly removing repeated code) while adding more coverage
  • feat(core): babel-transform-decorators-legacy removed from dependencies (no longer used in tests)
  • feat(core): recompose is now a dev dependency only (no longer used in HOCs as part of fix for bug(withFirebase): "Warning: Failed child context type..." when using react v16 #337)

Check List

If not relevant to pull request, check off as complete

  • All tests passing
  • Docs updated with any changes or examples if applicable
  • Added tests to ensure new feature(s) work properly

Relevant Issues

…337

* exposed reducer (matches common redux panther also seen with other
libraries such as redux-form)
@codecov
Copy link

codecov bot commented Dec 11, 2017

Codecov Report

Merging #358 into v2.0.0 will increase coverage by 1.8%.
The diff coverage is 92.51%.

@@            Coverage Diff            @@
##           v2.0.0     #358     +/-   ##
=========================================
+ Coverage   82.81%   84.61%   +1.8%     
=========================================
  Files          22       23      +1     
  Lines        2281     2386    +105     
  Branches      350      362     +12     
=========================================
+ Hits         1889     2019    +130     
+ Misses        392      367     -25

Scott Prue and others added 3 commits December 11, 2017 23:48
…tion. package-lock updated.

* dispatch added to withFirebase
* Fix double dispatch of FILE_UPLOAD_START when using progress: true
option
* transform-decorators-legacy removed (no longer used in tests)
@prescottprue prescottprue merged commit 44fc5ab into v2.0.0 Dec 17, 2017
@prescottprue prescottprue deleted the v2.0.0-rc.1 branch December 17, 2017 06:59
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.

2 participants