Skip to content

Feature/STC-746: Adapt meeting PDF exports for semantic identifiers - #23708

Merged
akabiru merged 1 commit into
devfrom
feature/stc-746-adapt-meeting-pdf-exports-for-semantic-identifiers
Jun 15, 2026
Merged

Feature/STC-746: Adapt meeting PDF exports for semantic identifiers#23708
akabiru merged 1 commit into
devfrom
feature/stc-746-adapt-meeting-pdf-exports-for-semantic-identifiers

Conversation

@akabiru

@akabiru akabiru commented Jun 12, 2026

Copy link
Copy Markdown
Member

Ticket

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

What are you trying to accomplish?

On instances configured with project-based work package identifiers, meeting PDF exports still showed the numeric database id. This PR makes the agenda and minutes templates render the configured identifier instead, so a work package agenda item exports as Task PROD-42 Improve onboarding flow rather than Task #39 Improve onboarding flow. The same applies to work package outcomes. Work package mentions inside agenda item notes already resolved to the configured identifier through the shared markdown-to-PDF pipeline, and the meeting exporter specs now cover that path too.

Screenshots

Before (classic identifiers)

stc746-minutes-classic-before

stc746-minutes-classic-before.pdf

After (semantic identifiers)

stc746-minutes-semantic-after

stc746-minutes-semantic-after.pdf

What approach did you choose and why?

The agenda title and outcome lines built their label with the raw numeric id, while the rest of the application (the meeting page itself, the work package PDF exports, mention expansion) renders identifiers through formatted_id, which returns the hash-prefixed numeric id in classic mode and the project-based identifier in semantic mode. The fix is to build the meeting PDF labels the same way, through a single helper shared by the agenda item title and the outcome line. Link URLs needed no change since URL helpers already resolve to the configured identifier.

Undisclosed work package references (where the viewer lacks permission) keep showing the numeric id, consistent with how the meeting page renders them.

Merge checklist

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

Agenda item titles and work package outcomes hardcoded the numeric id.
Using formatted_id renders the project-based identifier in semantic
mode, matching the meeting page rendering.
@github-actions

Copy link
Copy Markdown

Warning

Flaky specs

  • rspec ./spec/features/work_packages/details/inplace_editor/subject_editor_spec.rb[1:2:1:1:2]
  • rspec ./spec/features/work_packages/share/multi_invite_spec.rb[1:1:1]

@akabiru
akabiru requested a review from a team June 12, 2026 11:39
@akabiru
akabiru marked this pull request as ready for review 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 a94f34f into dev Jun 15, 2026
14 checks passed
@akabiru
akabiru deleted the feature/stc-746-adapt-meeting-pdf-exports-for-semantic-identifiers branch June 15, 2026 09:02
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants