Skip to content

Documentation/COMMS-897: Document target_versions in the incoming e-mail handler - #24361

Open
akabiru wants to merge 1 commit into
devfrom
documentation/comms-897-document-target-versions-incoming-email
Open

Documentation/COMMS-897: Document target_versions in the incoming e-mail handler#24361
akabiru wants to merge 1 commit into
devfrom
documentation/comms-897-document-target-versions-incoming-email

Conversation

@akabiru

@akabiru akabiru commented Jul 21, 2026

Copy link
Copy Markdown
Member

https://community.openproject.org/wp/COMMS-897

Follow-up to COMMS-897 (#24345), which taught the incoming e-mail handler to read and write target versions. That PR carries the code; this one splits out the documentation so it can be reviewed separately.

Updates docs/installation-and-operations/configuration/incoming-emails/README.md to document the new target_versions argument (IMAP_ATTR_TARGET_VERSIONS) and the target_versions: e-mail keyword, noting that several versions can be comma-separated and that more than one requires the multiple-versions feature. Also marks the legacy version / IMAP_ATTR_VERSION argument and version: keyword as deprecated in favour of target_versions.

Follow-up to #24345: document the new target_versions argument (IMAP_ATTR_TARGET_VERSIONS) and e-mail keyword, and mark the legacy version option as deprecated.
akabiru added a commit that referenced this pull request Jul 21, 2026
The README changes now live in #24361 so the documentation can be reviewed independently of this handler change.
@akabiru akabiru added this to the 17.8.x milestone Jul 21, 2026
@akabiru akabiru self-assigned this Jul 21, 2026
@akabiru
akabiru marked this pull request as ready for review July 21, 2026 10:38
@akabiru
akabiru requested review from a team and MayaBerd July 21, 2026 10:38
@github-actions

Copy link
Copy Markdown

Caution

The provided work package version does not match the core version

Details:

Please make sure that:

  • The work package version OR your pull request target branch is correct

@brunopagno brunopagno 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.

Approving from communicator side. This looks correct to me 😁

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants