Skip to content

Implementation/STC-783: Adapt PDF export of timesheets for semantic identifiers - #23712

Merged
akabiru merged 1 commit into
devfrom
implementation/stc-783-adapt-pdf-export-of-timesheets-for-semantic-identifiers
Jun 12, 2026
Merged

Implementation/STC-783: Adapt PDF export of timesheets for semantic identifiers#23712
akabiru merged 1 commit into
devfrom
implementation/stc-783-adapt-pdf-export-of-timesheets-for-semantic-identifiers

Conversation

@akabiru

@akabiru akabiru commented Jun 12, 2026

Copy link
Copy Markdown
Member

Ticket

https://community.openproject.org/wp/STC-783

What are you trying to accomplish?

On instances configured with project-based work package identifiers, the timesheet PDF export still showed the numeric database id in the work package column. The column now renders the configured identifier, so an entry exports as PROD-42 Improve onboarding flow rather than #65 Improve onboarding flow.

Screenshots

Before (classic identifiers)

stc783-timesheet-classic-before

stc783-timesheet-classic-before.pdf

After (semantic identifiers)

stc783-timesheet-semantic-after

stc783-timesheet-semantic-after.pdf

What approach did you choose and why?

Same approach as the meeting PDF exports (#23708): the work package cell built its label from the raw numeric id, while the rest of the application renders identifiers through formatted_id, which returns the hash-prefixed numeric id in classic mode and the project-based identifier in semantic mode. The cell label now goes through that method; classic-mode output is unchanged, and the link URL already resolved to the configured identifier through the URL helpers.

Merge checklist

  • Added/updated tests
  • Added/updated documentation in Lookbook (patterns, previews, etc)
  • Tested major browsers (Chrome, Firefox, Edge, ...)

The work package column hardcoded the numeric id. Using formatted_id
renders the project-based identifier in semantic mode.
@akabiru akabiru self-assigned this Jun 12, 2026
@akabiru akabiru added this to the 17.6.x milestone Jun 12, 2026
@akabiru
akabiru marked this pull request as ready for review June 12, 2026 11:21
@akabiru
akabiru requested a review from a team June 12, 2026 11:39

@thykel thykel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚀

@akabiru
akabiru merged commit ee8375d into dev Jun 12, 2026
16 of 17 checks passed
@akabiru
akabiru deleted the implementation/stc-783-adapt-pdf-export-of-timesheets-for-semantic-identifiers branch June 12, 2026 14:09
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Development

Successfully merging this pull request may close these issues.

2 participants