collectiveidea / awesome_nested_set

An awesome replacement for acts_as_nested_set and better_nested_set.
MIT License
2.39k stars 492 forks source link

allow order_for_rebuild to reference an associated table #396

Closed jakebrady5 closed 3 years ago

jakebrady5 commented 5 years ago

While this feature may be somewhat niche, I had a need for it. I thought I would PR the changes in case anyone else finds them useful.

The unscoped changes did not break any specs, but I do wonder about whether that might be a breaking change for some people. One upside from my perspective though, is that it gets closer to supporting other default scope gems like acts_as_paranoid.

Example Use Case

Using the spec models for consistency:

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.