community-platform icon indicating copy to clipboard operation
community-platform copied to clipboard

[feature request] Dont mention all contributors on research steps

Open davehakkens opened this issue 2 years ago • 6 comments

Is your feature request related to a problem? Please describe. Currently you can see to many contributors on a research step. You loose track of who is the creator and who just did some small grammar fixes.

Describe the solution you'd like Dont mention contributors that edit research steps only that create steps

Example of the problemt Is can be found in the research module here. Step 6 afbeelding

davehakkens avatar Apr 12 '23 18:04 davehakkens

I would challenge the assumption that subsequent edits are simple grammar changes. That may be true of the changes you've made in this specific case however the editor could have done anything to the content.

The current implementation stores the following information:

collaborators: ["username", "another-username"]

I propose the following change to the data structure:

collaborators: [
  {
     "userId": "username",
     "action": "created", // This could be either "created" or "edited",
     "datestamp": "2023-03-13T12:00:00.000Z"
  },
  {
     "userId": "another-username",
     "action": "edited",
     "datestamp": "2023-04-13T16:26:20.134Z"
  },
  {
     "userId": "username",
     "action": "edited",
     "datestamp": "2023-04-14T16:26:20.134Z"
  }
]

This allows us to store a more detailed record of the order of changes. For the reading view of the Research content we can filter this list down to only show the creator however we have the additional information available if needed. Maybe within the editor view we'd like to show a "last edited by username"

thisislawatts avatar Apr 13 '23 16:04 thisislawatts

Agree this would be a better and more complete solution. So show the creator on the frontend, store the data of contributors on the backend. To be figured out later how to best use it.

davehakkens avatar Apr 13 '23 18:04 davehakkens

Hey everyone, I'd like to give this is a go so long as it's still an open issue.

alexHoggett avatar Apr 26 '23 16:04 alexHoggett

Welcome @alexHoggett, that would be much appreciated. ✨ Let us know if you need any more information.

thisislawatts avatar Apr 26 '23 16:04 thisislawatts

Just checking in. How is this one going @alexHoggett ?

davehakkens avatar May 13 '23 16:05 davehakkens

Putting this one back to the pool @alexHoggett, if you want to carry on with it just let us know.

iSCJT avatar Jun 22 '23 14:06 iSCJT

:tada: This issue has been resolved in version 1.80.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

onearmy-bot avatar Jul 19 '23 13:07 onearmy-bot