Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make payload delimiter configurable #281

Merged
merged 6 commits into from
Aug 28, 2024
Merged

Make payload delimiter configurable #281

merged 6 commits into from
Aug 28, 2024

Commits on Jan 17, 2024

  1. Use underscore delimiter for flattening 

    The default key delimiter when flattening payloads is a period, which is not allowed in Slack workflow variable keys.
    @rzumer
    rzumer committed Jan 17, 2024
    Configuration menu
    Copy the full SHA
    2c6a605 View commit details
    Browse the repository at this point in the history

Commits on Jul 18, 2024

  1. Make the payload delimiter configurable 

    This reverts the default payload delimiter to a period and allows setting the `payload-delimiter` option to override it.
    @rzumer
    rzumer committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    1ebdac9 View commit details
    Browse the repository at this point in the history

Commits on Aug 28, 2024

  1. chore: merge w upstream main

    @zimeg
    zimeg committed Aug 28, 2024
    Configuration menu
    Copy the full SHA
    5cabcc8 View commit details
    Browse the repository at this point in the history

  2. docs: group details about the payload delimiter and defaults

    @zimeg
    zimeg committed Aug 28, 2024
    Configuration menu
    Copy the full SHA
    de895c6 View commit details
    Browse the repository at this point in the history

  3. test: confirm webhook payloads are flattened when sending to workflow… 

    … builder
    @zimeg
    zimeg committed Aug 28, 2024
    Configuration menu
    Copy the full SHA
    d82b331 View commit details
    Browse the repository at this point in the history

  4. docs: make it clear that delimiters are needed for just nested payloads

    @zimeg
    zimeg committed Aug 28, 2024
    Configuration menu
    Copy the full SHA
    9008c33 View commit details
    Browse the repository at this point in the history