Created an Individual Positions Detail Page by ACBerea · Pull Request #660 · BCStudentSoftwareDevTeam/lsf

ACBerea · 2026-07-30T16:03:55Z

Issue Description

Fixes Issue #611

Departments should be able to open any position from the Manage Positions page and view its complete position description.
This page serves as a view of a position and should display all information associated with the current version of the position.
Users should also be able to download the official Work-Learning-Service Position Description directly from this page.

Changes

Implemented the following requirements:

Create an Individual Position Details page.
The page displays the following in a group:
- Position Title
- Position Code
- WLS Level
- Position Status
- Last Revision Date
- Revision Author
The page displays the complete position description within a box, including:
- WLS Level Justification
- Description of Duties
- Learning Opportunities
- Required Qualifications
The Download button allows for downloading the position description document.
Displays whether the position is:
- Active
- Requested
- Inactive
Displays only the current version of the position.
Uses a back button to return to the Department Portal

Testing

Getting to the Individual Positions Detail Page

Select Department Portal on the sidebar.
Select a department.
On the Positions Card, select a position (highlighted in blue).
The page will reflect the positions information within the dummy data for Positions.

Downloading a Position Description PDF

To download a Position Description PDF, click on the Download Description Button.
Check your file explorer or download history for the PDF.

Back Button

On the Individual Positions Detail Page, click the Back button to return the Department Portal.

Rational for Implementation Decisions

The functions created for this only account for the current version of a position and not its history (this should be rectified in the issue Add Position History to Individual Position Page #612.
The demo data used does not fully reflect the implementation of the actual position description, as it does not exist as of creating this pull request.
The download description button logic should be updated when the accurate position description implementation is used.
Since no other type of position status aside from "Active" is currently in use, only "Active" positions are considered.
Database implementation cannot be completed until the proper position description data is provided.

…l and css for the page

…pencies.

…SoftwareDevTeam/lsf into individual_pos_page_JA

…a, and adjusted a few line of demp_data to provide more than one example of accurate data.

…dd the pdf format in download.py

MImran2002 · 2026-07-30T17:09:34Z

                           positions = positionsList,
                           posURL = posURL)

+@main_bp.route('/department/<org>/<account>/positions/<positionCode>', methods=['GET'])


this whole route should be in departmentportal.py route not in main_route

MImran2002 · 2026-07-30T17:11:54Z

                           posURL = posURL)

+@main_bp.route('/department/<org>/<account>/positions/<positionCode>', methods=['GET'])
+def individualPosition(org, account, positionCode):


instead of individualPosition it should be positionDescription as we are basically displaying the content of PDF

MImran2002 · 2026-07-30T17:12:15Z

+        position=position
+    )
+
+@main_bp.route('/department/<org>/<account>/positions/<positionCode>/download', methods=['GET'])


in departmentPortal.py route

MImran2002 · 2026-07-30T17:21:33Z

+    except (NameError, DoesNotExist):
+        return render_template('errors/404.html'), 404
+
+    position = PositionHistory.get_or_none(


turn this into one line and the positionHisotry.status="active" shouldn't be there as the issue description says: Display whether the position is:
-Active
-Requested
-Inactive

Notes:
This page displays only the current version. meaning we will display either active or requested or inactive but we will only show the current one meaning the most recent one. so your filter here should be the most recent instead of active filter. this mean if the most recent one is a requested status that should be shown.

MImran2002 · 2026-07-30T17:27:31Z


    return None

+def makePositionDescriptionPDF(department, position, revisionAuthor):


revisionAuthor is an extra parameter that is not use that mean you should also remove it from the function called in main routes: makePositionDescriptionPDF(dept, position, position.revisedBy) so remove the positions.revisedBY as the parameter revisionauthor is never used.

MImran2002 · 2026-07-30T17:33:42Z

+        ('Position Code', position.positionCode),
+        ('WLS Level', position.wls),
+        ('Status', position.status),
+        ('Last Revision Date', position.revisionDate),


create a test for the makepositionsdescriptionPDF you can find there are modules that read pdf and traverse and you can assert those.

…escription and downloadPositionDescription out of main_routes.py and into departmentPortal.py.

…nd accounts for revision dates. Added checks for revision dates to account for positions that have status' other than active in departmentPortal.py. Modified parameters in downloads.py.

… the new getPositionRevision function in the test_getPosition.py.

…instances accordingly. Began work on a test function for download,py

johnlolonga19 and others added 13 commits July 28, 2026 16:30

added revisedBy to positionHistory.py and created demo data for it

5cfa98c

added the routes for the individual position page and created the htm…

fb4924f

…l and css for the page

added the download position description button

bc35aa4

resizig html texts

64c729b

Fixed alignment issue with buttons caused by bootstrap version descre…

30030a2

…pencies.

Improved css namaing structure and began working on improved demo data.

cdc18b6

made minor to changes to css, html, and demo_data files

d8b4ece

Merge branch 'department-portal-base' of https://github.com/BCStudent…

f700635

…SoftwareDevTeam/lsf into individual_pos_page_JA

changes around the position descrription for format

e4fd883

Adjusted HTML in individualPositions.html for consistent formatting.

722606b

Fixed some demo data to accuratly reflect sudo-real data.

36e6d95

Fixed margin issue in individualPosition.html, replaced outdated jinj…

c0698e5

…a, and adjusted a few line of demp_data to provide more than one example of accurate data.

added functionality to the download button, by updating routing and a…

79c21f9

…dd the pdf format in download.py

ACBerea changed the base branch from development to department-portal-base July 30, 2026 17:05

MImran2002 requested changes Jul 30, 2026

View reviewed changes

ACBerea and others added 8 commits July 30, 2026 14:46

Renamed individualPosition to positionDescription and moved positionD…

ff37958

…escription and downloadPositionDescription out of main_routes.py and into departmentPortal.py.

Implemented a new function in getPositions to get a single position a…

21e6670

…nd accounts for revision dates. Added checks for revision dates to account for positions that have status' other than active in departmentPortal.py. Modified parameters in downloads.py.

Added a clarifying comment to positionHistory.py and added a test for…

85e2321

… the new getPositionRevision function in the test_getPosition.py.

Reverted getPositionRevision function to getPosition and updated all …

9268ad0

…instances accordingly. Began work on a test function for download,py

Added download.py test file missed in previous commit.

5c830c1

move the downoad button to the top right, and fixed pdf formatting

bb55ca8

added position description model

2f9bcda

added the sections logic and demo data

78e1152

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Created an Individual Positions Detail Page - #660

Created an Individual Positions Detail Page#660
ACBerea wants to merge 21 commits into
department-portal-basefrom
individual_pos_page_JA

ACBerea commented Jul 30, 2026

Uh oh!

MImran2002 Jul 30, 2026

Uh oh!

MImran2002 Jul 30, 2026

Uh oh!

MImran2002 Jul 30, 2026

Uh oh!

MImran2002 Jul 30, 2026

Uh oh!

MImran2002 Jul 30, 2026

Uh oh!

MImran2002 Jul 30, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants


		return None

		def makePositionDescriptionPDF(department, position, revisionAuthor):

Uh oh!

Conversation

ACBerea commented Jul 30, 2026

Issue Description

Changes

Testing

Rational for Implementation Decisions

Uh oh!

MImran2002 Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

MImran2002 Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

MImran2002 Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

MImran2002 Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

MImran2002 Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

MImran2002 Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants