Fixes a problem with deleting an object and it not being deleted out of an association #35

Merged
merged 1 commit into from Apr 28, 2012

Conversation

Projects
None yet
2 participants
@sheuer
Contributor

sheuer commented Apr 27, 2012

No description provided.

Make sure properties with indexes are deleted properly
This fixes a problem with deleting an object and it not being deleted
out of an association

whoahbot added a commit that referenced this pull request Apr 28, 2012

Merge pull request #35 from sheuer/master
Fixes a problem with deleting an object and it not being deleted out of an association

@whoahbot whoahbot merged commit cba6291 into whoahbot:master Apr 28, 2012

@whoahbot

This comment has been minimized.

Show comment Hide comment
@whoahbot

whoahbot Apr 28, 2012

Owner

Thank you very much for the patch and specs! I merged in your changes and
pushed up a new gem version, v0.6.1. I also added you to the README so that
your name can be heralded in song by all those that use your work.

whoahbot

On Fri, Apr 27, 2012 at 1:32 PM, Stephen Heuer <
reply@reply.github.com

wrote:

You can merge this Pull Request by running:

git pull https://github.com/sheuer/dm-redis-adapter master

Or you can view, comment on it, or merge it online at:

#35

-- Commit Summary --

  • Make sure properties with indexes are deleted properly

-- File Changes --

M lib/dm-redis-adapter/adapter.rb (4)
M spec/dm_redis_associations_spec.rb (41)

-- Patch Links --

https://github.com/whoahbot/dm-redis-adapter/pull/35.patch
https://github.com/whoahbot/dm-redis-adapter/pull/35.diff


Reply to this email directly or view it on GitHub:
#35

Owner

whoahbot commented Apr 28, 2012

Thank you very much for the patch and specs! I merged in your changes and
pushed up a new gem version, v0.6.1. I also added you to the README so that
your name can be heralded in song by all those that use your work.

whoahbot

On Fri, Apr 27, 2012 at 1:32 PM, Stephen Heuer <
reply@reply.github.com

wrote:

You can merge this Pull Request by running:

git pull https://github.com/sheuer/dm-redis-adapter master

Or you can view, comment on it, or merge it online at:

#35

-- Commit Summary --

  • Make sure properties with indexes are deleted properly

-- File Changes --

M lib/dm-redis-adapter/adapter.rb (4)
M spec/dm_redis_associations_spec.rb (41)

-- Patch Links --

https://github.com/whoahbot/dm-redis-adapter/pull/35.patch
https://github.com/whoahbot/dm-redis-adapter/pull/35.diff


Reply to this email directly or view it on GitHub:
#35

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