Skip to content

Conversation

@toolstack
Copy link
Contributor

No description provided.

Copy link
Owner

@akirk akirk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar to #63 I really appreciate the work and idea to add support for this but I don't see storing this in user meta as sustainable. Can we also do this as a filter for now so that you can get this functionality running until we have an implementation using taxonomies?

@toolstack toolstack marked this pull request as draft January 15, 2024 20:02
@toolstack
Copy link
Contributor Author

Converted to terms.

@toolstack toolstack marked this pull request as ready for review January 16, 2024 02:08
@mediaformat
Copy link
Contributor

This would be great, anything holding this back @toolstack ?

@toolstack
Copy link
Contributor Author

Not that I know of other than it looks like it needs an update to resolve the conflicts.

@toolstack
Copy link
Contributor Author

Took a closer look, looks like somethings have changed (like the permissions) that need to be updated in the PR as well.

@akirk is there any interest in actually merging this? If so I'll make the updates.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants