Feature proposal #8664
closed
Improve several core asset usages
100%
Description
Several core activity assets are being poorly used. Mainly commented on and liked type activity. For example when a photo is uploaded to CB Gallery then it being a single photo means comments, likes, etc.. SHOULD be attached directly to the photo. This means commenting on it and generating a commented on activity entry should ALSO be attached directly to the photo. This however isn't the case. Example as follows.
groupjive.group.GROUP_ID.gallery.photos.PHOTO_ID
This is a gallery photo upload to a GJ group. Now commenting on it would generate the following comment activity.
activity.ACTIVITY_ID
This is WRONG. It should be as follows.
groupjive.group.GROUP_ID.gallery.photos.PHOTO_ID.comment
This is a continuation of #8009 where it was decided major structure rewrites aren't necessary and that several core assets are fine (e.g. blog assets).
Updated by krileon over 3 years ago
Several object verbs should probably also be improved. Example as follows.
groupjive.group.GROUP_ID.gallery.photos.PHOTO_ID.comment
Changed to the following.
groupjive.group.GROUP_ID.gallery.photos.PHOTO_ID.comment.COMMENT_ID
This would let us know what comment is associated with the activity. Might not be possible to migrate the existing usages though.
Updated by krileon about 3 years ago
Plugin assets could use some improvements as well. Specifically CB GroupJive and CB Gallery. These can be safely updated to the following.
FROM: groupjive.group.GROUP_ID
TO: group.GROUP_ID
FROM: gallery.photos.PHOTO_ID
TO: gallery.MEDIA_ID
It's not necessary to prefix GJ and it's not necessary to have type for CB Gallery. This would give shorter assets where possible. Example as follows.
OLD: groupjive.group.GROUP_ID.gallery.photos.PHOTO_ID
NEW: group.GROUP_ID.gallery.MEDIA_ID
Updated by krileon about 3 years ago
Change profile.PROFILE_ID to user.USER_ID. This will prepare the asset for future standalone extension usage.
Updated by krileon about 3 years ago
Change the long "connection" verb to just "conn".
Updated by krileon over 1 year ago
- Status changed from Assigned to Rejected
- Target version deleted (
6.0.0) - % Done changed from 0 to 100
This isn't really necessary now with store streams and better handling of asset between objects.