Skip to content

General: Add localized gender suffixes to job titles - #2641

Draft
ge94zec wants to merge 2 commits into
mainfrom
general/2640-add-mwd-in-title
Draft

General: Add localized gender suffixes to job titles#2641
ge94zec wants to merge 2 commits into
mainfrom
general/2640-add-mwd-in-title

Conversation

@ge94zec

@ge94zec ge94zec commented Aug 24, 2026

Copy link
Copy Markdown
Contributor
  • display (m/w/d) in German and (m/f/d) in English
  • show localized suffixes in job forms, cards, and detail views
  • include suffixes in applicant and professor PDF exports
  • keep stored job titles and exported filenames unchanged

Checklist

General

Server

  • Important: I implemented the changes with a very good performance and prevented too many (unnecessary) and too complex database calls.
  • I strictly followed the principle of data economy for all database calls.
  • I strictly followed the server coding and design guidelines.
  • I added multiple server tests (JUnit) related to the features (with a high test coverage), while following the server test guidelines.
  • I documented the Java code using JavaDoc style.

Client

  • Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data (e.g. using paging).
  • I strictly followed the principle of data economy for all client-server REST calls.
  • I strictly followed the client coding and design guidelines.
  • I added multiple integration tests (Vitest) related to the features (with a high test coverage), while following the client test guidelines.
  • I documented the TypeScript code using JSDoc style.
  • I added multiple screenshots/screencasts of my UI changes.

Motivation and Context

Description

Steps for Testing

Prerequisites:

  1. Log in to DocApply

Review Progress

Code Review

  • Code Review 1

Manual Tests

  • Test 1

Screenshots

Test Coverage

Client

Class/File Line Coverage Lines Expects Ratio
job-creation-form.component.ts 0.00% 1308 103 7.9
job-detail.component.ts 0.00% 498 59 11.8
job-card.component.ts 0.00% 83 8 9.6
base-input.component.ts 0.00% 120 ? ?
string-input.component.ts 0.00% 31 16 51.6
pdf-labels.ts 100.00% 125 9 7.2
job-title.util.ts 100.00% 5 1 20.0

Server

Class/File Line Coverage Lines
PDFExportService.java 74.86% 515

Last updated: 2026-08-24 20:01:57 UTC

- display (m/w/d) in German and (m/f/d) in English
- show localized suffixes in job forms, cards, and detail views
- include suffixes in applicant and professor PDF exports
- keep stored job titles and exported filenames unchanged
@ge94zec ge94zec linked an issue Aug 24, 2026 that may be closed by this pull request
@github-actions github-actions Bot added server Pull requests that update Java code. (Added Automatically!) client Pull requests that update TypeScript code. (Added Automatically!) tests labels Aug 24, 2026
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 4 complexity

Metric Results
Complexity 4

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@ge94zec ge94zec changed the title feat: add localized gender suffixes to job titles General: Add localized gender suffixes to job titles Aug 24, 2026
- remove compliance error matching from the job title
- keep compliance analysis limited to the job description
@github-actions

Copy link
Copy Markdown
Contributor

🤖 No OpenAPI or client changes needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client Pull requests that update TypeScript code. (Added Automatically!) server Pull requests that update Java code. (Added Automatically!) tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add (m/w/d) in title

1 participant