Skip to content

Feat/room capacities#91

Open
SumanB1432 wants to merge 3 commits intometakgp:masterfrom
SumanB1432:feat/room-capacities
Open

Feat/room capacities#91
SumanB1432 wants to merge 3 commits intometakgp:masterfrom
SumanB1432:feat/room-capacities

Conversation

@SumanB1432
Copy link
Copy Markdown

Description

This PR adds a new roomCapacities.json data file under frontend/src/data/ that provides (dummy) seating capacity values for rooms used in Chillzone.

These capacities are consumed by the updated EmptyRoomsTable component to power the "minimum capacity" filter, so users can filter empty rooms not just by time and building but also by an approximate room size. The current values are reasonable estimates and can be refined later with accurate data.

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Manually verified the Empty Rooms view in the frontend:
    • Started the app with pnpm run dev.
    • Opened the Empty Rooms table.
    • Applied different "Min capacity" values and confirmed the room list changes accordingly.

Test Configuration:

  • OS: macOS
  • Tooling: pnpm, Node.js
  • SDK: N/A

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas (not required here as this is data-only)
  • I have made corresponding changes to the documentation (optional: can be a follow-up PR)
  • My changes generate no new warnings
  • I have added tests that prove my feature works (manual testing only for this data change)
  • New and existing unit tests pass locally with my changes (if applicable)
  • Any dependent changes have been merged and published in downstream modules (N/A for this data-only change)

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