Skip to content

Conversation

@MatthewDailey
Copy link
Contributor

Description

This PR fixes a couple minor errors in the type declaration for react-redux-firebase. They're all very simple fixes and look like minor typos or missed spots when bulk updating types.

Screenshot of the errors fixed:
Screen Shot 2020-11-30 at 5 11 49 PM

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

#997

@codecov
Copy link

codecov bot commented Dec 1, 2020

Codecov Report

Merging #1031 (325bbe6) into master (c72de9d) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1031      +/-   ##
==========================================
+ Coverage   88.33%   88.44%   +0.11%     
==========================================
  Files          29       29              
  Lines         797      805       +8     
==========================================
+ Hits          704      712       +8     
  Misses         93       93              

@MatthewDailey MatthewDailey changed the title fix(typescript): Fix several minor type errors fix(typescript): fix several minor type errors Dec 1, 2020
@4sp3ll
Copy link

4sp3ll commented Jan 2, 2021

I have huge issue with this bug:

Duplicate identifier 'endBefore'. TS2300
444 | */
445 | where?: WhereOptions | WhereOptions[]

446 | endBefore?: FirestoreTypes.DocumentSnapshot | any | any[]
| ^
447 | /**
448 | * @see https://github.com/prescottprue/redux-firestore#orderby
449 | */

When bugfix will be released? Or there is any other way to fix it?

@prescottprue prescottprue changed the base branch from master to v3.9.0 January 4, 2021 22:26
@prescottprue prescottprue changed the title fix(typescript): fix several minor type errors fix(typescript): fix reloadAuth argument type and duplicate of endBefore Jan 4, 2021
@prescottprue prescottprue merged commit 3b80135 into prescottprue:v3.9.0 Jan 4, 2021
@prescottprue prescottprue mentioned this pull request Jan 4, 2021
3 tasks
prescottprue added a commit that referenced this pull request Jan 5, 2021
* fix(auth): prevent throw in `logout` when database is not setup (#1042) - @ssdns
* fix(types): fix `reloadAuth` argument type and duplicate of `endBefore` (#1031) - @ MatthewDailey
* fix(types): fix argument type for `login` with credentials (#1045) - @AlexanderArvidsson
* fix(types): add back `FirestoreStatics` to `ExtendedFirestoreInstance` (#1030)
* fix(core): add deprecation message `enableLogging` config option
* chore(docs): add note about required rules for profile settings (#1049)

Co-authored-by: Matthew Dailey <[email protected]>
Co-authored-by: Alexander Arvidsson <[email protected]>
Co-authored-by: shishido <[email protected]>
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