Skip to content

[MA-583] Busyness Card Refresh Bug Fix#2293

Closed
aleyu0 wants to merge 3 commits intoUCSD:experimentalfrom
aleyu0:busyness-card-fix
Closed

[MA-583] Busyness Card Refresh Bug Fix#2293
aleyu0 wants to merge 3 commits intoUCSD:experimentalfrom
aleyu0:busyness-card-fix

Conversation

@aleyu0
Copy link
Copy Markdown

@aleyu0 aleyu0 commented Mar 9, 2026

Summary

Dining halls category on the busyness card keeps being untoggled upon refresh.

  • Preserves the location view state across availability refreshes, even when ⁠selectedOccuspaceLocations is null or empty.

  • Prevents crashes in the availability pager when no locations are visible or when the page controller has no clients.
  • Re‑enables Markets by no longer filtering it out (temporary disable removed).

Changelog

[General] [Fix] - Preserve busyness card location selection across refresh and prevent crashes when no locations are visible

Test Plan

  1. Dining halls stay selected across refresh
  • Without logging in, add busyness location categories, then refresh the card
    • all selected cards should persist
  • Log in with a user and add busyness location categories
    • refresh should maintain all active categories
    • closing and reopening app should maintain all active categories
  1. No selected locations
  • Untoggle all categories the busyness card.

  • Check card to ensure expected behavior
  • Refresh the card and should still keep no cards
  1. Locations data availability
  • With updated JSON, should see the following locations (check names as they are now using an alias)
    • Libraries: Geisel, Wong Avery
    • Price Center
    • Recreational Facilities: RIMAC Gym, Main Gym, Triton Esports Center, Outdoor Climbing Center
    • Dining Halls: OceanView, Pines, 64 Degrees, Ventanas, Canyon Vista Marketplace, Club Med, Restaurants at Sixth College
    • Markets: Roger's Market, Seventh Market, Sixth Market

@aleyu0 aleyu0 force-pushed the busyness-card-fix branch from 0a50571 to 7a00c18 Compare March 9, 2026 20:00
@aleyu0 aleyu0 closed this Mar 11, 2026
@aleyu0
Copy link
Copy Markdown
Author

aleyu0 commented Mar 11, 2026

changes were added to #2287

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