Skip to content

Conversation

@jakobwesthoff
Copy link

In order to archive compatibility with the current PouchDB release (6.3.4), the following changes were applied:

  • Certain adapters (like http) have a dedicated put implementation now, which does not use bulkDocs any more, therefore a transformation needs to be applied in this case.
  • The latest pouchdb-wrappers implementation hasn't been handling the TaskQueue correctly. Therefore a race condition could case the transformation methods to be applied twice (Wait for TaskQueue to be ready before calling wrapper chain pouchdb/pouchdb-wrappers#3). Until the corresponding PR is merged and a new release is issued. I linked the dependency directly to the PR branch. I will try and update the PR once a new pouchdb-wrappers release is issued.
  • The pouchdb dependencies for testing were bumped to 6.3.4.

Until the corresponding PR is merged into the project, this branch should do the trick. It is a version 1.3.6 + the race condition fix.

Once the PR is merged and a new release is issued this needs to be switched back to the real realease.
@gr2m
Copy link
Collaborator

gr2m commented Jul 21, 2017

can you have a look why the tests with Phantom are failing right now? Might be related to 6bdaebc? There is also another issue around Promise: #47

@jakobwesthoff
Copy link
Author

Fixed. The new version of pouchdb-wrappers utilizes native Promises, which PhantomJS does not support. The addition of lie as a polyfill fixed the issue.

Copy link
Collaborator

@gr2m gr2m left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Let's get pouchdb/pouchdb-wrappers#3 merged and released first, so we don’t need to install from github, okay?

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