GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||
GeezerVison is showing github.com — fix(logs): restore narrow v1 detail query by TheodoreSpeaks · Pull Request #6588 · simstudioai/sim Conversation
Contributor
Summary
Type of Change
Testing
Checklist
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
|
There was an error while loading. Please reload this page.
PR SummaryLow Risk Overview The inline select joins only Reviewed by Cursor Bugbot for commit 3ca8818. Bugbot is set up for automated code reviews on this repo. Configure here. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Greptile SummaryThis PR restores a route-specific query for v1 log details while preserving the endpoint’s existing authorization and response behavior.
Confidence Score: 5/5The PR appears safe to merge with no actionable correctness or security regressions identified. The narrowed query preserves every field used by the response, handles deleted workflows through the existing nullable join behavior, and retains the authentication and workspace authorization gates.
|
| Filename | Overview |
|---|---|
| apps/sim/app/api/v1/logs/[id]/route.ts | Replaces the broad shared log query with a narrow projection while retaining all fields consumed by the route and the existing workspace-access check. |
Reviews (1): Last reviewed commit: "fix(logs): restore narrow v1 detail quer..." | Re-trigger Greptile
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
3ca88183
Deployed
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.