Skip to content

Risk repository validator: Add back button and improve coding layout - #70

Merged
rgambee merged 5 commits into
mainfrom
rr-validator/back-button-and-coding-layout
Sep 1, 2026
Merged

rgambee merged 5 commits into
mainfrom
rr-validator/back-button-and-coding-layout

Conversation

@rgambee

@rgambee rgambee commented Sep 1, 2026

Copy link
Copy Markdown
Member

This makes a few changes as requested in Slack:

  1. Use the assignee's email if they don't have a name set in Airtable
    • Previously papers would be grouped under Unassigned, which is wrong
  2. Add a back button to return from a single paper back to the list
  3. Improve the coding layout on small screens
    • Shrink comment boxes and the sidebar to make more room for the important content

Warn the user if they're leaving behind unsaved changes.
This leaves more room for the risk details above the controls.
Previously, the sidebar would extend to cover the full window with no
way to hide it, making the app unusable on narrow viewports.
It's still present, unlike for small windows, but in a more compact
form.
@vercel

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
airi-pipelines Ready Ready Preview Sep 1, 2026 10:45pm UTC

Request Review

@rgambee
rgambee merged commit bcba026 into main Sep 1, 2026
4 checks passed
@rgambee
rgambee deleted the rr-validator/back-button-and-coding-layout branch September 1, 2026 22:47
rgambee added a commit that referenced this pull request Sep 23, 2026
…70)

This makes a few changes as requested in Slack:

1. Use the assignee's email if they don't have a name set in Airtable
- Previously papers would be grouped under Unassigned, which is wrong
2. Add a back button to return from a single paper back to the list
3. Improve the coding layout on small screens
- Shrink comment boxes and the sidebar to make more room for the
important content

This branch was successfully deployed

1 active deployment
Preview — a5739630 Deployed Sep 1, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant