Skip to content

Update Array Map Processor screenshots and fix grammar in processor docs#38034

Merged
pavel-kim-dd merged 5 commits into
masterfrom
pavel.kim/array-map-processor-screenshot-updates
Jul 9, 2026
Merged

Update Array Map Processor screenshots and fix grammar in processor docs#38034
pavel-kim-dd merged 5 commits into
masterfrom
pavel.kim/array-map-processor-screenshot-updates

Conversation

@pavel-kim-dd

Copy link
Copy Markdown
Contributor

What does this PR do?

  • Updates string builder and arithmetic sub-processor screenshots for the Array Map Processor doc
  • Fixes "replaces ... by" → "replaces ... with" in arithmetic_processor.md and string_builder_processor.md

@github-actions github-actions Bot added the Images Images are added/removed with this PR label Jul 8, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

@pavel-kim-dd pavel-kim-dd marked this pull request as ready for review July 8, 2026 16:40
@pavel-kim-dd pavel-kim-dd requested a review from a team as a code owner July 8, 2026 16:40
@janine-c janine-c self-assigned this Jul 8, 2026
@janine-c janine-c added the editorial review Waiting on a more in-depth review label Jul 8, 2026

@janine-c janine-c 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.

Looks fantastic, thanks for applying this so quickly, Pavel! I made the same small suggestion in a couple places to make the phrasing a little more consistent, but that's super tiny 🙂

| `expression` | String | Yes | Arithmetic operation between one or more log attributes. |
| `target` | String | Yes | Name of the attribute that contains the result of the arithmetic operation. |
| `is_replace_missing` | Boolean | No | If `true`, it replaces all missing attributes of `expression` by 0, `false` skip the operation if an attribute is missing. Default: `false`. |
| `is_replace_missing` | Boolean | No | If `true`, it replaces all missing attributes of `expression` with 0. If `false`, skips the operation if an attribute is missing. Default: `false`. |

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.

Suggested change
| `is_replace_missing` | Boolean | No | If `true`, it replaces all missing attributes of `expression` with 0. If `false`, skips the operation if an attribute is missing. Default: `false`. |
| `is_replace_missing` | Boolean | No | If `true`, replaces all missing attributes of `expression` with 0. If `false`, skips the operation if an attribute is missing. Default: `false`. |

| `template` | String | Yes | A formula with one or more attributes and raw text. |
| `target` | String | Yes | The name of the attribute that contains the result of the template. |
| `is_replace_missing` | Boolean | No | If `true`, it replaces all missing attributes of `template` by an empty string. If `false`, skips the operation for missing attributes. Default: `false`. |
| `is_replace_missing` | Boolean | No | If `true`, it replaces all missing attributes of `template` with an empty string. If `false`, skips the operation for missing attributes. Default: `false`. |

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.

Suggested change
| `is_replace_missing` | Boolean | No | If `true`, it replaces all missing attributes of `template` with an empty string. If `false`, skips the operation for missing attributes. Default: `false`. |
| `is_replace_missing` | Boolean | No | If `true`, replaces all missing attributes of `template` with an empty string. If `false`, skips the operation for missing attributes. Default: `false`. |

@janine-c janine-c removed their assignment Jul 8, 2026
@janine-c janine-c removed the editorial review Waiting on a more in-depth review label Jul 8, 2026
@pavel-kim-dd

Copy link
Copy Markdown
Contributor Author

Looks fantastic, thanks for applying this so quickly, Pavel! I made the same small suggestion in a couple places to make the phrasing a little more consistent, but that's super tiny 🙂

@janine-c Thanks, applied the fixes!

@pavel-kim-dd pavel-kim-dd merged commit 299da82 into master Jul 9, 2026
17 checks passed
@pavel-kim-dd pavel-kim-dd deleted the pavel.kim/array-map-processor-screenshot-updates branch July 9, 2026 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Images Images are added/removed with this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants