search

opensafely-core / airlock

Other
2 stars 0 forks source link

Update css classes for combined file decision status #506

Closed rebkwok closed 2 months ago

rebkwok commented 2 months ago

Fixes #489 Agreed reviews: 2 reviewers have approved: Screenshot from 2024-07-10 16-54-40

Agreed reviews: 2 reviewers have requested changes: Screenshot from 2024-07-10 16-55-16

Conflicted reviews: Current reviewer has approved, other reviewer has requested changes: Screenshot from 2024-07-10 16-55-37

Author's view of conflicted file in request: Screenshot from 2024-07-10 17-11-31

Author's view of under-review file in workspace (this is unchanged): Screenshot from 2024-07-10 17-13-23

Author's view of updated file in workspace (this is unchanged): image

After returning and resubmitting (with no changes, so previous request-changes decisions are now undecided) - shows the user review status: image

Review has now approved and completed review (other reviewer has rejected and not yet completed): image

Other reviewer has now completed, shows combined decision: image

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

Deploying airlock-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: c4168dc
Status: ✅  Deploy successful!
Preview URL: https://8ec3cfdb.airlock.pages.dev
Branch Preview URL: https://file-decision-classes.airlock.pages.dev

View logs