Skip to content

Community Library miscelaneous fixes - #6076

Open
AlexVelezLl wants to merge 2 commits into
learningequality:hotfixesfrom
AlexVelezLl:esocc-miscelaneous-fixes
Open

Community Library miscelaneous fixes#6076
AlexVelezLl wants to merge 2 commits into
learningequality:hotfixesfrom
AlexVelezLl:esocc-miscelaneous-fixes

Conversation

@AlexVelezLl

Copy link
Copy Markdown
Member

Summary

  • The admin Channels table reset its status filter (e.g. "Needs review") when navigating back to it, since an unconditional reset watcher fired on every mount instead of only when the status became genuinely invalid for the selected type.
  • The community library status button had a fixed width that clipped longer labels.

Reviewer guidance

To manually verify the channel table fix: in the admin Channels table, filter by "Community Library" + "Needs review", open a channel, then use the browser Back button — the filters should stay as set instead of resetting.

Before After
image image

AI usage

Used Claude Code throughout: diagnosed the channel-table filter-reset bug via code exploration, implemented and iterated on the fix. I reviewed all diffs, decided the final fix approach for the filter bug, and ran the test suite and pre-commit before this PR.

AlexVelezLl and others added 2 commits August 3, 2026 07:43
…table

Navigating back to the admin Channels table (e.g. after opening a
channel and using the browser back button) reset the status filter
even though it was correctly restored from the URL. An immediate
watcher unconditionally overwrote it to the first available option on
every mount instead of only defaulting when the filter is actually
unset. Since the status filter is derived from the current channel
type's options, a status no longer valid for a new type already reads
back as unset, so a single "is it set" check covers both the
type-change reset and back-navigation preservation.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The fixed width clipped longer status labels. Let it size to content.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@AlexVelezLl
AlexVelezLl changed the base branch from unstable to hotfixes August 3, 2026 12:55
@AlexVelezLl
AlexVelezLl requested a review from rtibbles August 3, 2026 12:56
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