Skip to content

refactor: LocationState management#90

Merged
rrrockey merged 10 commits intomainfrom
refactor/location-state-management
Feb 11, 2026
Merged

refactor: LocationState management#90
rrrockey merged 10 commits intomainfrom
refactor/location-state-management

Conversation

@rrrockey
Copy link
Copy Markdown
Collaborator

@rrrockey rrrockey commented Feb 11, 2026

Screen.Recording.2026-02-11.at.11.45.42.mov

This PR includes various fixes to Employees, Tasks, and the LocationState service.

Changes are listed below:

/users

  • Update the admins' Manage Users page to:
    • only show users within their company
    • only show locations within their company
    • correctly assign a location to a user

/tasks

  • Fix generic tasks not showing up for employees

Nav Bar

  • Remove My Tasks view from managers' and admins' Nav Bar
  • Remove Inventory view from employees’ navBar

Miscellaneous

  • Update the admins' /settings/locations page to properly add users to a location
  • Fix app freezing and not updating for employee dashboard when an invalid API key is assigned
  • Minor UI updates

Known Issues:

  • LocationState is still not persistent on refresh, the app is persistent across pages, and will show info from that location everywhere, but will go back to the first location when the app is reloaded.
    • Will address this in another PR, but wanted to get these changes banked.

@rrrockey rrrockey changed the title Refactor/location-state-management refactor: LocationState management Feb 11, 2026
Copy link
Copy Markdown
Collaborator

@kevbang kevbang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great, good job! :)

@rrrockey rrrockey merged commit ca808a0 into main Feb 11, 2026
1 check failed
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.

2 participants