I'd like to add the ability for user's to comment on various activities they (or others) have performed. But I'm not sure how to do this in FarmOS - there doesn't appear to be Views available from Admin > Structure, and not content types.

Comments

BruceDawson created an issue. See original summary.

m.stenta’s picture

This is a great feature request! And one that has been discussed a little bit in the past (I think Dorn had the idea originally to provide some form of conversations on things). My plan for implementing this has been to enable Comments on our custom entity types: Farm Asset, and Log.

Unfortunately, in Drupal 7, the Comment module only supports nodes. This has been re-architected in Drupal 8, and now Comments can be added to any entity type. So this feature will probably have to wait until farmOS is updated for Drupal 8 - something I am eager to get rolling on soon!

m.stenta’s picture

In the meantime, the only way to do it is to edit the Activity log itself and just add more text to the "Notes" field. Obviously not ideal, but out-of-the-box that's the only way to do it.

You could also add a custom multivalue text field, and have your users edit and just add new entries with that field. But that would just be a temporary solution, and you would have to write your own upgrade path to Drupal 8 when the time comes.

kadaan’s picture

Any reason we can't use comments_entity In Drupal7?

m.stenta’s picture

@kadaan - We could, but I'm inclined to say let's wait until Drupla 8 - where we will get comments on all entity types for free.

If we add comments_entity in farmOS 7.x-1.x, it means we also need to write the upgrade path to automatically migrate them to farmOS 8.x-2.x.

I think this will be a great feature in 8.x-2.x, but I'd like to keep things simple in 7.x-1.x so we can get there as soon as possible. In the meantime, advanced users can certainly try the comments_entity module themselves - but I don't think we should take on the responsibility of supporting it.

kadaan’s picture

@mstenta: I'm fine with that. We should figure out how I can best accelerate the migration to 8.x

m.stenta’s picture

Version: 7.x-1.0-beta12 » 2.x-dev
m.stenta’s picture

Status: Active » Closed (outdated)
Related issues: +#3352183: Comments on farmOS records

I opened a dedicated issue for farmOS 2.x: #3352183: Comments on farmOS records