Add event emitters to models and emit some interesting events. #10

Merged
merged 8 commits into from Apr 7, 2013

Conversation

Projects
None yet
1 participant
@ceejbot
Owner

ceejbot commented Apr 7, 2013

Also, finish off the levelup adapter by implementing the last attachments features.

ceejbot added some commits Apr 7, 2013

Switched to emitting events for before/after um events instead of
testing for functions then calling them.

I'm using LucidJS as an event library just in case I ever try to make
this work in the browser.

Added unit tests for all triggered events.

Wrote a mock db and moved several tests from the couch adapter suite
to the generic persistence layer suite. There's more work to be done
there.

Bumped the version number in anticipation of release.
The levelup adapter now cleans up attachments properly.
Added unit tests to prove it. Or so I hope.

The default polyclay require no longer includes all the db adapters, so
you do not need to have them all built in order to use the package.
Right now you have to require the path inside the package directly.
I'll find some better way to do that.
Emit change events when attachments are changed.
Added unit tests for the events.

ceejbot added a commit that referenced this pull request Apr 7, 2013

Merge pull request #10 from ceejbot/evented
Add event emitters to models and emit some interesting events.

@ceejbot ceejbot merged commit 5db4090 into master Apr 7, 2013

1 check passed

default The Travis build passed
Details

@ceejbot ceejbot deleted the evented branch Apr 7, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment