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

pouchdb-validation #247

Merged
merged 28 commits into from
Jul 4, 2017
Merged

pouchdb-validation #247

merged 28 commits into from
Jul 4, 2017

Conversation

gr2m
Copy link
Contributor

@gr2m gr2m commented Jul 3, 2017

part of #197

marten-de-vries and others added 28 commits July 3, 2017 15:57
…d to the registry if necessary. For now just there's a script that links them locally (and installs all their dependencies).
…he plug-ins (the largest is now 4.1kb min+gzip). Added JS Coverage info. Some new tests.
…dapter. Validation doesn't occur offline if the server handles it. No test suite regressions, but both are still experimental. (And JS coverage is lower.)
… A few bug fixes. Code coverage to 100% (python, js); no more fixmes/todos. Pretty much only pouchdb.mapping is now left as experimental.
…x to the nodejs test suite, some documentation changes.
… much untested). Also validation update (will become 1.1.0).
… some improvements to coroutines (nesting them, most notably)
…& validation wrappers. Untested additions to the security plug-in (but no regressions).
…rotections for them like in CouchDB). No regressions, but JS test coverage isn't complete yet.
a.k.a. making it play nice with Kneden.
@nolanlawson nolanlawson mentioned this pull request Jul 3, 2017
12 tasks
validate_doc_update functions on the client side. A browser version is
available.

See also [pouchdb-validation's documentation](http://pythonhosted.org/Python-PouchDB/js-plugins.html#pouchdb-validation-plug-in)
Copy link
Member

Choose a reason for hiding this comment

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

Can we point this to our mono-repo. Or mention it is part of the mono-repo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I plan to do a post-monorepo-merge README cleanup, there are several things that need to be done

Copy link
Member

@garrensmith garrensmith left a comment

Choose a reason for hiding this comment

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

Nice work

@gr2m gr2m merged commit 256af44 into master Jul 4, 2017
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.

None yet

3 participants