Skip to content

v2.0.0 beta.12 #298

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

Merged
merged 15 commits into from
Oct 24, 2017
Merged

v2.0.0 beta.12 #298

merged 15 commits into from
Oct 24, 2017

Conversation

prescottprue
Copy link
Owner

@prescottprue prescottprue commented Oct 19, 2017

Description

  • feat(core): recompose used for withFirebase and withFirestore HOCs
  • feat(core): promiseEvents method added. Similar to firebaseConnect, but not a React Higher Order Component - feat(core): non-HOC version of firebaseConnect, for use with server side rendering #299 - thanks @marekolszewski
  • feat(core): redux-firestore is no longer included and is now an optionalDependency (increased bundle size too much)
  • feat(helpers): getVal added to simplify migration from v1 to v2 (can easily replace dataToJS) - Add getValueAt function for easy migration to v2 from dataToJS and pathToJS #305 - thanks @fej-snikduj
  • feat(reducer): added error to clearly indicate wrong initial state being passed to errors state
  • feat(reducer): preserve now supports passing an object to preserve auth, profile, errors, and ordered
  • feat(docs): Firestore setup instructions switched to using redux-firestore externally
  • feat(examples): firestore example uses redux-firestore directly (since it is no longer included)
  • fix(typings): remove an artifact from a previous merge conflict - Remove an artifact from a previous merge conflict #304 - thanks @cbellino
  • fix(reducer): authError state is now null by default (tests changed to match)
  • fix(examples): examples all point to next tag for consistency
  • fix(examples): snippets README.md files updated
  • fix(examples): tests folder removed from material example

Potentially Breaking

  • authError state is now null by default instead of {} - this is more clear, and more closely resembles v1.*.*

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

@codecov
Copy link

codecov bot commented Oct 19, 2017

Codecov Report

Merging #298 into v2.0.0 will decrease coverage by 0.02%.
The diff coverage is 84.42%.

@@            Coverage Diff             @@
##           v2.0.0     #298      +/-   ##
==========================================
- Coverage    81.3%   81.27%   -0.03%     
==========================================
  Files          20       22       +2     
  Lines        2011     2083      +72     
  Branches      315      322       +7     
==========================================
+ Hits         1635     1693      +58     
- Misses        376      390      +14

marekolszewski and others added 11 commits October 19, 2017 22:20
…ide rendering (#299)

* Added a non-HOC version of firebaseConnect, for use with server side rendering.

* PR updates based on comments.
…301

* authError is set to null on LOGOUT and LOGIN actions
* LOGIN_ERROR no longer affects profile state - #301
…JS (#305)

* Add getValueAt function

* Change name of data to firebase

* Remove regex

* Add example

* Add default value at rename getValueAt to getVal
…ded.

* authError state is now `null` by default
* compose.md file removed from docs folder (old and replaced by
enhancer.md)
* preserve supported added to auth, profile, errors, and ordered states
@prescottprue prescottprue merged commit 006a22c into v2.0.0 Oct 24, 2017
@prescottprue prescottprue deleted the v2.0.0-beta.12 branch October 24, 2017 06:47
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.

3 participants