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

Cascades #13

Open
RustyDev opened this issue Jan 11, 2016 · 2 comments
Open

Cascades #13

RustyDev opened this issue Jan 11, 2016 · 2 comments
Labels

Comments

@RustyDev
Copy link

I've noticed a few things about cascading when deleting various models.

If a user likes a post and either the user or the post is deleted, the likes aren't deleted as well and the counts don't update.

@rtconner rtconner added the bug label Jan 18, 2016
@JesusLeon
Copy link

It would be definitely nice if this is in the package out-of-the-box. But I think it relies more on the application logic.

@RustyDev you could easily add an observer to update the likes data in that scenario. But of course it would be even better if you could send a PR with such functionality :)

@rtconner
Copy link
Owner

rtconner commented Feb 7, 2016

I haven't had to add a config file to this library yet, but if I add this feature I'll want to put in a config file for it. To make the cascade optional.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants