Skip to content

PLU-591: [show-multiple-rows-3]: add preprocess to postman email#1538

Open
m0nggh wants to merge 1 commit intofeat/find-multiple-rows/table-var-regex-markerfrom
feat/find-multiple-rows/table-var-postman
Open

PLU-591: [show-multiple-rows-3]: add preprocess to postman email#1538
m0nggh wants to merge 1 commit intofeat/find-multiple-rows/table-var-regex-markerfrom
feat/find-multiple-rows/table-var-postman

Conversation

@m0nggh
Copy link
Copy Markdown
Contributor

@m0nggh m0nggh commented Apr 10, 2026

TL;DR

Added support for table variables in Postman transactional email body with HTML table rendering and error handling.

What changed?

  • Added preprocessVariable function to the send transactional email action that converts table variable markers to HTML tables
  • Implemented type guard isTableMarker to identify table variable markers
  • Restricted table variable usage to the email body field only, showing placeholder text for other fields
  • Added graceful error handling that displays error messages instead of failing the entire step
  • Updated the body field configuration to accept table variables alongside existing variable types
  • Extended IFieldRichText interface to support variableTypes specification

How to test?

  1. Create a workflow with a Postman send transactional email step
  2. Use a table variable in the email body field and verify it renders as an HTML table
  3. Try using a table variable in other fields (subject, recipient) and confirm placeholder text appears
  4. Test with malformed table data to ensure error messages display properly instead of step failure
  5. Verify existing variable types (text, array, etc.) continue to work as expected

Why make this change?

This enhancement allows users to include structured tabular data in their transactional emails, making it easier to send formatted reports, lists, or other structured information. The restriction to body-only usage and graceful error handling ensures the feature is both useful and robust.

Copy link
Copy Markdown
Contributor Author

m0nggh commented Apr 10, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add the label lfg to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@m0nggh m0nggh changed the title add preprocess to postman email PLU-591: [show-multiple-rows-3]: add preprocess to postman email Apr 10, 2026
@linear
Copy link
Copy Markdown

linear bot commented Apr 10, 2026

@m0nggh m0nggh marked this pull request as ready for review April 10, 2026 13:15
@m0nggh m0nggh requested a review from a team as a code owner April 10, 2026 13:15
@m0nggh m0nggh force-pushed the feat/find-multiple-rows/table-var-regex-marker branch from a618a12 to 1f9edd2 Compare April 10, 2026 13:53
@m0nggh m0nggh force-pushed the feat/find-multiple-rows/table-var-postman branch from 9d59663 to 15e0687 Compare April 10, 2026 13:53
@m0nggh m0nggh force-pushed the feat/find-multiple-rows/table-var-regex-marker branch from 1f9edd2 to df14166 Compare April 11, 2026 02:11
@m0nggh m0nggh force-pushed the feat/find-multiple-rows/table-var-postman branch from 15e0687 to ce7775d Compare April 11, 2026 02:11
@m0nggh m0nggh force-pushed the feat/find-multiple-rows/table-var-postman branch from ce7775d to 6d4c268 Compare April 15, 2026 02:47
@m0nggh m0nggh force-pushed the feat/find-multiple-rows/table-var-regex-marker branch from df14166 to 62ac196 Compare April 15, 2026 02:47
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