Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(tests): Test forwardWebhook #3424

Merged
merged 3 commits into from
Feb 3, 2025
Merged

fix(tests): Test forwardWebhook #3424

merged 3 commits into from
Feb 3, 2025

Conversation

nalanj
Copy link
Contributor

@nalanj nalanj commented Jan 31, 2025

The test file for forwardWebhook was testing sendAuth instead. This fixes that.

@nalanj nalanj self-assigned this Jan 31, 2025
@nalanj nalanj marked this pull request as ready for review January 31, 2025 19:29
@nalanj nalanj requested a review from a team January 31, 2025 19:29
Copy link
Collaborator

@bodinsamuel bodinsamuel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What a weird issue :D

@nalanj nalanj enabled auto-merge (squash) February 3, 2025 19:14
@nalanj nalanj disabled auto-merge February 3, 2025 19:14
@nalanj nalanj merged commit a0f4a31 into master Feb 3, 2025
17 checks passed
@nalanj nalanj deleted the alan/forward-webhook-test branch February 3, 2025 19:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants