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

Fix "db.type() is deprecated and will be removed in a future version of PouchDB" #3925

Closed
SCdF opened this issue Sep 19, 2017 · 5 comments
Closed
Labels
Priority: 3 - Low Can be bumped from the release Type: Technical issue Improve something that users won't notice

Comments

@SCdF
Copy link
Contributor

SCdF commented Sep 19, 2017

The above is getting splatted into the console.

Work out why we're getting it and change our usage of PouchDB to avoid it.

@SCdF SCdF added Priority: 3 - Low Can be bumped from the release Type: Technical issue Improve something that users won't notice labels Sep 19, 2017
@garethbowen garethbowen self-assigned this Dec 5, 2017
@garethbowen
Copy link
Member

This is coming from worker-pouch - I've raised an issue and submitted a PR: pouchdb-community/worker-pouch#43

@garethbowen garethbowen removed their assignment Dec 5, 2017
garethbowen added a commit that referenced this issue Nov 21, 2018
This moves the cleanup function outside of the startup sequence
and also solves warning from PouchDB when trying to cleanup before
the db is initialised.

#3925
@garethbowen garethbowen mentioned this issue Nov 21, 2018
5 tasks
SCdF pushed a commit that referenced this issue Nov 21, 2018
This moves the cleanup function outside of the startup sequence
and also solves warning from PouchDB when trying to cleanup before
the db is initialised.

#3925
@SCdF SCdF removed their assignment Nov 21, 2018
@SCdF SCdF added Status: 4 - Acceptance testing and removed Priority: 3 - Low Can be bumped from the release labels Nov 21, 2018
@garethbowen garethbowen added Priority: 3 - Low Can be bumped from the release and removed Status: 3 - Code review labels Nov 21, 2018
@ngaruko
Copy link
Contributor

ngaruko commented Jan 7, 2019

Does this need any AT from QA @garethbowen ?

@garethbowen
Copy link
Member

Yes. Make sure the issue is no longer occurring.

@ngaruko
Copy link
Contributor

ngaruko commented Jan 7, 2019

LGTM
before
image

after
image

@ngaruko ngaruko closed this as completed Jan 7, 2019
@sookwalinga
Copy link

Seems this issue has again resurfaced in version 4.x, I am testing on v4.2.0 and I get the same warning: "db.type() is deprecated and will be removed in a future version of PouchDB"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 3 - Low Can be bumped from the release Type: Technical issue Improve something that users won't notice
Projects
None yet
Development

No branches or pull requests

4 participants