opensafely-core / airlock

Other
2 stars 0 forks source link

Hide votes and comments from output-checkers/users as needed #452

Closed bloodearnest closed 2 months ago

bloodearnest commented 3 months ago

Fixes #327

This is not the final version, but it gets the basic structure for tracking vote/comment visibility in place.

There will be some UI finesse needed - exposing the icons in the tree to the author, more info and details about which output checker voted which way.

Also, we need to decide what the consolidation of comments look like, so currently all comments remain private, although output checkers can create a public comment if they wasnt to. But that's different issue really, so left TODO in this change.

cloudflare-workers-and-pages[bot] commented 2 months ago

Deploying airlock-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 44c382b
Status: ✅  Deploy successful!
Preview URL: https://f3d022ff.airlock.pages.dev
Branch Preview URL: https://visibility.airlock.pages.dev

View logs

bloodearnest commented 2 months ago

Becky and I did an extensive walkthrough of this PR in a call together.

Here's my TODO list from that discussion: