Skip to content

SeamWebhook: normalize headers before passing to svix + add docstrings - #113

Merged
andrii-balitskyi merged 13 commits into
mainfrom
normalize-webhook-headers
Jul 12, 2024
Merged

SeamWebhook: normalize headers before passing to svix + add docstrings#113
andrii-balitskyi merged 13 commits into
mainfrom
normalize-webhook-headers

Conversation

@andrii-balitskyi

@andrii-balitskyiandrii-balitskyi commented Jul 10, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

Closes#84

@andrii-balitskyiandrii-balitskyi changed the title Normalize headers before passing to svix + add docstrings to SeamWebhook classSeamWebhook: normalize headers before passing to svix + add docstringsJul 10, 2024
razor-x
razor-x reviewed Jul 10, 2024
View reviewed changes
Comment threadseam/seam_webhook.py Outdated
Comment threadseam/seam_webhook.py Outdated
andrii-balitskyi
andrii-balitskyi commented Jul 11, 2024
View reviewed changes
Comment threadseam/__init__.py Outdated
razor-x
razor-x approved these changes Jul 11, 2024
View reviewed changes
Comment threadseam/seam_webhook.py Outdated
Comment threadseam/seam_webhook.py Outdated
Comment threadseam/seam_webhook.py Outdated
@andrii-balitskyi
andrii-balitskyi merged commit d816702 into mainJul 12, 2024
@andrii-balitskyi
andrii-balitskyi deleted the normalize-webhook-headers branch July 12, 2024 13:13
@DebbieAtSeam

DebbieAtSeam commented Jul 12, 2024

Copy link
Copy Markdown

@andrii-balitskyi or @razor-x There are three items with added docstrings here. Is there a code sample that shows each of these three items in use (in context)? I'm asking because I'm not yet familiar with working with webhooks programmatically for Seam. Thanks!

@razor-x

razor-x commented Jul 15, 2024

Copy link
Copy Markdown
Member

@DebbieAtSeamhttps://github.com/seamapi/python?tab=readme-ov-file#webhooks

This was referenced Jul 17, 2024
Closed
Merged
This was referenced Aug 1, 2024
Merged
Merged
Merged
Open
This was referenced Feb 12, 2025
Merged
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@razor-xrazor-xrazor-x approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Normalize headers to lowercase keys before passing to svix

4 participants

@andrii-balitskyi@DebbieAtSeam@razor-x@seambot