Skip to content

[OPIK-5665] [FE] fix: allow collapsing Ollie sidebar in error state#6159

Merged
awkoy merged 1 commit intomainfrom
awkoy/OPIK-5665-ollie-sidebar-error-collapse
Apr 10, 2026
Merged

[OPIK-5665] [FE] fix: allow collapsing Ollie sidebar in error state#6159
awkoy merged 1 commit intomainfrom
awkoy/OPIK-5665-ollie-sidebar-error-collapse

Conversation

@awkoy
Copy link
Copy Markdown
Contributor

@awkoy awkoy commented Apr 9, 2026

Details

When the compute endpoint returns 500 and the error state is shown, the Ollie sidebar was stuck open with no way to close it. This adds toggle functionality so the error state can be collapsed/expanded, matching the normal sidebar behavior.

  • Added onToggle to AssistantErrorState — expanded error shows an X close button, collapsed error strip expands on click
  • Made isOpen stateful in AssistantSidebarLoader so it can toggle, persisting to localStorage and updating layout width

Issues

https://comet-ml.atlassian.net/browse/OPIK-5665

Change checklist

  • I have reviewed my own code
  • I have added relevant tests or verified manually

Testing

  • Force a compute 500 (hardcode DEV_RESULT to error state) and verify expanded error shows X button
  • Click X — sidebar collapses to thin strip with error icon
  • Click collapsed strip — sidebar expands back to full error view
  • Retry button still works from expanded state
  • Collapse state persists across page reloads
  • Normal (non-error) sidebar loading/ready states unaffected

Documentation

No documentation changes needed.

When the compute endpoint returns 500 and the error state is shown,
the sidebar was stuck open with no way to close it. Add toggle
functionality so the error state can be collapsed/expanded, matching
the normal sidebar behavior.
@awkoy awkoy requested a review from a team as a code owner April 9, 2026 13:55
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

📋 PR Linter Failed

Missing Section. The description is missing the ## Details section.


Missing Section. The description is missing the ## Change checklist section.


Missing Section. The description is missing the ## Issues section.


Missing Section. The description is missing the ## Testing section.


Missing Section. The description is missing the ## Documentation section.

@awkoy awkoy merged commit 5c49cd0 into main Apr 10, 2026
10 of 11 checks passed
@awkoy awkoy deleted the awkoy/OPIK-5665-ollie-sidebar-error-collapse branch April 10, 2026 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants