Skip to content

Keep empty mapped fields out of the ticket body#368

Merged
ramereth merged 2 commits into
masterfrom
ramereth/fix-empty-custom-field-body
Jun 17, 2026
Merged

Keep empty mapped fields out of the ticket body#368
ramereth merged 2 commits into
masterfrom
ramereth/fix-empty-custom-field-body

Conversation

@ramereth

Copy link
Copy Markdown
Member

extract_custom_fields added a form field to the consumed set only after
confirming it had a value, so a mapped-but-empty field (e.g. an unfilled
SSH key) was never marked consumed and still appeared in the ticket body
as a blank line. Mark any well-formed mapping entry consumed before the
empty-value check, so the field is omitted from the body whether or not
it has a value; only the custom-field assignment is skipped when empty.

Co-Authored-By: Claude Opus 4.8 [email protected]
Signed-off-by: Lance Albertson [email protected]

ramereth and others added 2 commits June 16, 2026 17:26
extract_custom_fields added a form field to the consumed set only after
confirming it had a value, so a mapped-but-empty field (e.g. an unfilled
SSH key) was never marked consumed and still appeared in the ticket body
as a blank line. Mark any well-formed mapping entry consumed before the
empty-value check, so the field is omitted from the body whether or not
it has a value; only the custom-field assignment is skipped when empty.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
Signed-off-by: Lance Albertson <[email protected]>
Add Python 3.14 to the CI matrix so the test suite runs on it alongside
3.12 and 3.13. The runtime image already runs on python:3.14-alpine, so
this keeps CI aligned with what ships.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
@ramereth ramereth merged commit d60e5c3 into master Jun 17, 2026
7 checks passed
@ramereth ramereth deleted the ramereth/fix-empty-custom-field-body branch June 17, 2026 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant