storm icon indicating copy to clipboard operation
storm copied to clipboard

Add path enumerable trait

Open bennothommo opened this issue 2 years ago • 20 comments

Implements path enumeration hierarchy for models, operating similar to nested set models but not requiring the rigid left/right values.

This is in preparation for some work towards database storage of media items.

bennothommo avatar Aug 31 '23 06:08 bennothommo

This pull request will be closed and archived in 3 days, as there has been no activity in the last 60 days. If this is still being worked on, please respond and we will re-open this pull request. If this pull request is critical to your business, consider joining the Premium Support Program where a Service Level Agreement is offered.

github-actions[bot] avatar Dec 02 '23 01:12 github-actions[bot]

@bennothommo 👀

LukeTowers avatar Dec 07 '23 07:12 LukeTowers

@bennothommo let me know when you want me to review this!

LukeTowers avatar Dec 11 '23 20:12 LukeTowers

@LukeTowers it's good to go :)

bennothommo avatar Dec 11 '23 23:12 bennothommo

@bennothommo is there a docs PR for this?

LukeTowers avatar Jan 26 '24 15:01 LukeTowers

@LukeTowers not yet - I'm toying with the idea of unifying the tree traits in some fashion so they share a common base level of functionality and API.

bennothommo avatar Jan 27 '24 04:01 bennothommo

@bennothommo is that something that needs to happen now or could we move forward with merging this so that we can get metadata support added to the Media Library?

LukeTowers avatar Jan 27 '24 17:01 LukeTowers

@LukeTowers my concern is that we'll have difficulty changing the API later.

bennothommo avatar Jan 28 '24 11:01 bennothommo

This pull request will be closed and archived in 3 days, as there has been no activity in the last 60 days. If this is still being worked on, please respond and we will re-open this pull request. If this pull request is critical to your business, consider joining the Premium Support Program where a Service Level Agreement is offered.

github-actions[bot] avatar Apr 19 '24 01:04 github-actions[bot]

@bennothommo status on this?

LukeTowers avatar Apr 25 '24 23:04 LukeTowers

Just waiting for docs. I mentioned previously trying to attempt to unify the tree APIs, but I probably won't do it for this branch. It might be worth looking into next major release.

bennothommo avatar Apr 26 '24 00:04 bennothommo

@bennothommo sounds good to me, lets get the docs done up so we can roll this into the next release then. Would be nice to make some progress on the media library metadata.

LukeTowers avatar Apr 26 '24 03:04 LukeTowers

This pull request will be closed and archived in 3 days, as there has been no activity in the last 60 days. If this is still being worked on, please respond and we will re-open this pull request. If this pull request is critical to your business, consider joining the Premium Support Program where a Service Level Agreement is offered.

github-actions[bot] avatar Jun 26 '24 01:06 github-actions[bot]

@bennothommo ping

LukeTowers avatar Jul 22 '24 04:07 LukeTowers