Skip to content

Labor position review page - #669

Draft
nahom70 wants to merge 83 commits into
department-portal-basefrom
labor_position_review_page
Draft

Labor position review page#669
nahom70 wants to merge 83 commits into
department-portal-basefrom
labor_position_review_page

Conversation

@nahom70

@nahom70 nahom70 commented Aug 7, 2026

Copy link
Copy Markdown

This Pull request addresses a page for labor to view pending position approvals for review and edit, then approve them.

Progress so far:

  • Created a button on the action tab on the manage departments page for position review
  • The button navigates to all requested position reviews from that specific department through a query on the database
  • The review button then follows with a two-split section page where the left column is the old position description and the right is an editable part of the new position.
  • The right part of the page isn't implemented in a way to retrieve the new revised position from the department. For now, it is a duplicate of the left column ( old position )

Fixes needed:

  • The review page for a position needs to get the newly requested position description
  • The editable sections need to be able to send the data to the database when approved
  • In the case of saving ( not submitting the approval ), it should be able to save the state of the progress.
  • Lastly, making the page look good and consistent with the styling as it is on the figma file.

zhytkovd and others added 30 commits July 2, 2026 20:56
…rmName to academicYear in manageDepartments.html
…website. Added commented out code for the pending variation of the Approved Button in manageDepartments.html.
…will now change based on if the allocation isFinal from either pending or approved.
zhytkovd and others added 29 commits July 22, 2026 13:33
…eDepartments' into dep_portal_ad_ManageDepartments
…eDepartments' into dep_portal_ad_ManageDepartments
…r own files + Cleaned up the manageDepartments.js file
@nahom70
nahom70 marked this pull request as draft August 7, 2026 14:54
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.

5 participants