Wire Current Allocation card to real backend data (#602)#636
Open
DanielRukwasha wants to merge 4 commits into
Open
Wire Current Allocation card to real backend data (#602)#636DanielRukwasha wants to merge 4 commits into
DanielRukwasha wants to merge 4 commits into
Conversation
…602) Fix migrate_db.sh missing the Allocation model, seed demo Allocation rows per department, and query real allocated/used position counts and break hours in departmentPortal() instead of hardcoded placeholder numbers.
…-card-real-data-ruk # Conflicts: # app/controllers/main_routes/main_routes.py # app/templates/main/departmentPortal.html # database/demo_data.py # database/migrate_db.sh
MImran2002
requested changes
Jul 8, 2026
MImran2002
left a comment
Contributor
There was a problem hiding this comment.
your branch has conflict and is this final PR or draft PR
Author
|
@MImran2002 this is the final PR, not a draft. The merge conflict is resolved now too — branch is up to date with allocation_card and mergeable/clean. Ready for another look whenever you get a chance. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related to #602
Replaces the hardcoded placeholder numbers on the Department Portal's 'Current Allocation' card with real data:
Not yet done / out of scope for this PR: the 'Manage Allocations' button still points nowhere (destination page depends on other open issues), and there's no access-control check restricting a department's allocation data to Labor admins + that department's own staff.
Tested locally end-to-end in the devcontainer against real seeded data across multiple departments.