Skip to content

Support SQS message group ID#139

Open
kokobd wants to merge 2 commits into
Nike-Inc:masterfrom
bellroy:sqs-message-group-id-upstream
Open

Support SQS message group ID#139
kokobd wants to merge 2 commits into
Nike-Inc:masterfrom
bellroy:sqs-message-group-id-upstream

Conversation

@kokobd
Copy link
Copy Markdown

@kokobd kokobd commented Feb 23, 2026

  • Added messageGroupId to attributes of SQSEvent.
  • Added a unit test for parsing SQS events.

* Support SQS message group id

* derive Generic instances

* use ApplicativeDo

* haddock; changelog
@kokobd kokobd requested a review from IamfromSpace as a code owner February 23, 2026 19:25
@endgame
Copy link
Copy Markdown
Contributor

endgame commented Feb 23, 2026

CI failures are due to using an old version of the cache action.

@IamfromSpace
Copy link
Copy Markdown
Collaborator

This looks good! I'll try to get this building and releasing soon.

@kokobd
Copy link
Copy Markdown
Author

kokobd commented Mar 17, 2026

Hello, we realized later that messageGroupId is an optional field. It's only provided by AWS if the queue is a FIFO queue. I've changed the type to Maybe Text.

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.

3 participants