Radish alpha
r
rad:z4D5UCArafTzTQpDZNQRuqswh3ury
Radicle desktop app
Radicle
Git
Show additional information in patch list view
Open rudolfs opened 1 year ago

We’re missing the following data on the list_patches and patch_by_id commands for the Patch type:

  • total comment count (across all revisions and reviews)
  • review count
    • we could also use a summary of all verdicts or just show each review verdict separately together with the author alias + avatar
z6MkkfM3...sVz5 commented 1 year ago

I agree with the review count, and should be pretty straight forward to add, maybe only for the latest revision?

Not so sure about the total comment count over all revisions and reviews, what would be the benefit of that number? See if a patch is trending?

rudolfs commented 1 year ago

The total comment count gives some indication on how active a discussion is I guess. We have that counter for issues, so felt weird not to have it on patches. Sometimes I remember roughly how much discussion there was on something and if I see a bigger number scrolling through a list I’d likely check out what’s happening there. And sometimes comments get burried under Rebase revisions, so it might be good to know the total count.