Add a Product asset type and Product type taxonomy #787
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mstenta
added a commit
to mstenta/farmOS
that referenced
this pull request
Feb 2, 2024
mstenta
force-pushed
the
3.x-product-asset
branch
from
d89172e
to
86c0539
Compare
paul121
approved these changes
Feb 2, 2024
There was a problem hiding this comment.
I'm approving given that it is inevitable we will have a product asset type.
I share some of the concerns/issues raised in the forum topic linked with more context/discussion around how effectively this can be used without more dedicated UIs for managing products and inventory. I hope the product asset leads to more solutions than conceptual issues/questions for users but alas, power to the people.
mstenta
added a commit
to mstenta/farmOS
that referenced
this pull request
Feb 2, 2024
mstenta
force-pushed
the
3.x-product-asset
branch
from
86c0539
to
351a5bc
Compare
mstenta
force-pushed
the
3.x-product-asset
branch
from
351a5bc
to
a37820e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I originally started this in a separate module (https://github.com/mstenta/farm_product), but I think we should consider adding it to farmOS core as a standard asset type.
Inspired by this forum topic today: https://farmos.discourse.group/t/asset-type-product/1864
For more discussion/context see: https://farmos.discourse.group/t/tracking-products-and-inventory-product-quantity/1613