Skip to content

Conversation

@cruzdanilo
Copy link
Contributor

Description

fix options not being passed to updateProfile action

@codecov
Copy link

codecov bot commented May 17, 2019

Codecov Report

Merging #701 into next will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             next     #701   +/-   ##
=======================================
  Coverage   82.98%   82.98%           
=======================================
  Files          30       30           
  Lines         964      964           
  Branches      168      168           
=======================================
  Hits          800      800           
  Misses        164      164

@prescottprue prescottprue changed the title pass updateProfile options to action fix(auth): pass updateProfile options to action Jul 1, 2019
@prescottprue prescottprue changed the base branch from next to v2.3.0 July 1, 2019 20:54
@prescottprue
Copy link
Owner

Going to make a separate commit to avoid conflicts with master as I put this into an upcoming latest release. Will also get that out to next sometime soon.

@prescottprue prescottprue changed the base branch from v2.3.0 to next July 1, 2019 20:56
@prescottprue prescottprue mentioned this pull request Jul 1, 2019
3 tasks
@prescottprue prescottprue changed the base branch from next to 3.0.0-alpha.13 July 13, 2019 04:05
@prescottprue prescottprue changed the title fix(auth): pass updateProfile options to action fix(profile): pass updateProfile options to action Jul 13, 2019
@prescottprue prescottprue merged commit ac5d18b into prescottprue:3.0.0-alpha.13 Jul 13, 2019
@prescottprue prescottprue mentioned this pull request Jul 13, 2019
3 tasks
prescottprue added a commit that referenced this pull request Jul 13, 2019
* feat(types): add types for hooks including `useFirestore`, `useFirebase`, `useFirestoreConnect`, `useFirebaseConnect` - #653
* fix(types): make scopes optional in `credentials` - #671
* fix(docs): remove v2 auth ready docs from v3 docs - #676
* fix(providers): prevent unwanted re-initialization when parent component updates - #725 - @illuminist
* fix(profile): pass `updateProfile` options to action - #701 - @cruzdanilo 
* fix(firebaseConnect): remove `dispatch` from reserved props list - #675, #700
* feat(types): improve types for `withFirestore` and `withFirebase`
* feat(types): added `ExtendedFirebaseInstance` and `ExtendedFirestoreInstance` types
* feat(examples): update typescript example to use hooks
* fix(firestoreConnect): add error for passing of reserved props `firebase` and `firestore` when using `firestoreConnect`
* feat(types): add typing for firebase queries used in `firebaseConnect` and `useFirebaseConnect`
* fix(types): fix spelling of initialize in types
* feat(docs): add api docs pages for context providers and new hooks api 
* feat(docs): automatically find files for doc generation instead of having a constant that needs updating
* feat(core): add uglify and lodash plugins to webpack build to shrink bundle build size
@prescottprue
Copy link
Owner

Included in v3.0.0-alpha.13 release. The change is also planned to go out into latest with v2.3.0. Thanks for the PR!

prescottprue added a commit that referenced this pull request Jul 13, 2019
* feat(storage): pass file metadata through `uploadFile` to `storage.put` calls - #720
* fix(auth): pass `updateProfile` options to action - #701 - @cruzdanilo
* fix(profile): only include `providerData` if it is not an empty array in Firestore - #699
* feat(webpack): add `lodash-webpack-plugin` to shrink bundle size
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