Skip to content

Conversation

yosizelensky
Copy link
Contributor

Closes #5319

πŸ“‘ Description

βœ… Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

β„Ή Additional Information

@vercel
Copy link

vercel bot commented Sep 17, 2025

@yosizelensky is attempting to deploy a commit to the KeepHQ Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Sep 17, 2025
@CLAassistant
Copy link

CLAassistant commented Sep 17, 2025

CLA assistant check
All committers have signed the CLA.

@dosubot dosubot bot added Bug Something isn't working Provider Providers related issues labels Sep 17, 2025
shahargl
shahargl previously approved these changes Sep 17, 2025
Copy link
Member

@shahargl shahargl left a comment

Choose a reason for hiding this comment

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

lgtm

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 17, 2025
@vercel
Copy link

vercel bot commented Sep 17, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
keep Ignored Ignored Preview Sep 21, 2025 10:58am

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Sep 21, 2025
@codecov
Copy link

codecov bot commented Sep 21, 2025

Codecov Report

βœ… All modified and coverable lines are covered by tests.
βœ… Project coverage is 46.45%. Comparing base (57ca737) to head (f3fc2ea).
⚠️ Report is 19 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5320      +/-   ##
==========================================
- Coverage   46.64%   46.45%   -0.19%     
==========================================
  Files         175      176       +1     
  Lines       18263    18385     +122     
==========================================
+ Hits         8519     8541      +22     
- Misses       9744     9844     +100     

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

πŸš€ New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • πŸ“¦ JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@shahargl shahargl enabled auto-merge (squash) September 21, 2025 11:30
Copy link
Member

@shahargl shahargl left a comment

Choose a reason for hiding this comment

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

Lgtm

@shahargl shahargl merged commit cfcf490 into keephq:main Sep 21, 2025
17 of 19 checks passed
@github-actions
Copy link
Contributor

πŸš€ Fantastic work @yosizelensky! Your very first PR to keep has been merged! πŸŽ‰πŸ₯³

You've just taken your first step into open-source, and we couldn't be happier to have you onboard. πŸ™Œ
If you're feeling adventurous, why not dive into another issue and keep contributing? The community would love to see more from you! πŸš€

For any support, feel free to reach out on the community: https://slack.keephq.dev. Happy coding! πŸ‘©β€πŸ’»πŸ‘¨β€πŸ’»

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working lgtm This PR has been approved by a maintainer Provider Providers related issues size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[πŸ› Bug]: Webhook provider uses dummy payload for validation

3 participants