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

PLUGIN: couchDB not working #112

Open
GalinhaLX opened this issue Dec 6, 2022 · 7 comments
Open

PLUGIN: couchDB not working #112

GalinhaLX opened this issue Dec 6, 2022 · 7 comments

Comments

@GalinhaLX
Copy link

Hi

Sorry if it's not the right place to post for plugin issues

couchDB plugin is not working

when adding mv-plugins="couchdb" to mavo
browser says:

Couldn't find the requested file /mavo-couchdb.js in valterkraemer/mavo-couchdb.

it was looking for

"https://cdn.jsdelivr.net/gh/valterkraemer/mavo-couchdb@latest/mavo-couchdb.js"
"https://cdn.jsdelivr.net/gh/valterkraemer/mavo-couchdb/mavo-couchdb.js"

Also tried to load it manually, with a mavo-couchdb.js local file
using this file

but then got an error like

image

@LeaVerou LeaVerou transferred this issue from mavoweb/mavo Dec 6, 2022
@LeaVerou
Copy link
Member

LeaVerou commented Dec 6, 2022

Hmmm, it looks like this plugin has not been maintained so it doesn't seem to work with recent versions of Mavo.

@valterkraemer I know you’re not actively working on this anymore, but would you be open to merging PRs?

@valterkraemer
Copy link
Contributor

Hmmm, it looks like this plugin has not been maintained so it doesn't seem to work with recent versions of Mavo.

@valterkraemer I know you’re not actively working on this anymore, but would you be open to merging PRs?

Sure!

@LeaVerou
Copy link
Member

LeaVerou commented Dec 8, 2022

@DmitrySharabin any interest in submitting a PR? Converting $.Class() syntax to a regular class is pretty straightforward usually.

@DmitrySharabin
Copy link
Member

@DmitrySharabin any interest in submitting a PR? Converting $.Class() syntax to a regular class is pretty straightforward usually.

Sure! Will try to submit a PR asap.

@DmitrySharabin
Copy link
Member

Done!

valterkraemer/mavo-couchdb#2

@DmitrySharabin
Copy link
Member

@GalinhaLX It should be fixed now. Could you please check it?

@GalinhaLX
Copy link
Author

Hi

Sorry to delete previous answers, but I preferred to make it readable:

  • mavo-couch plugin works

-- It just needs to put mv-plugin="couchdb" code, that will load PouchDB and PouchDB-authentication automatically if needed

Except for:

-- Needs to press manually "save" button to save data
-- @LeaVerou , should this issue be re-opened?

  • Password.windows prompt ... displays password in readable text,

-- I'm not sure it's encrypted before being sent :(
-- Would it be possible to use some solutions discussed here to improve it?

  • Performance:
    @valterkraemer , I see
    1 mavo app => 1 couchDB database
    1 collection with multiple properties => 1 document (with all inside)!

in case of a huge list, CouchDB needs to load and parse all full list, apply filters before displaying it

Wouldn't it be possible to use the feature of couchDB called "Partitioned Database" to split properties that are inside a collection, into different documents inside CouchDB?

From CouchDB documentation:

A partitioned database requires a partition key for every document, where the document _id format is '<partition_key>:<doc_key>'. A partition is a logical grouping of documents. Partition queries are often faster than global ones.

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

No branches or pull requests

4 participants