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.

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

Update webhooks log count #2533

Merged
merged 1 commit into from
Nov 13, 2023
Merged

Update webhooks log count #2533

merged 1 commit into from
Nov 13, 2023

Conversation

nick-f
Copy link
Contributor

@nick-f nick-f commented Oct 20, 2023

Only 20 webhook request and responses can be viewed in the UI. Updating the docs to match reality.

@mbelton-buildkite
Copy link
Contributor

Hi @nick-f, thanks for finding this and fixing it! Sorry, it's taken so long for us to get to but I'll help get it resolved. First, I'll get someone from the team to verify the information is correct, then I'll run the linting and merge for you.

@dabarrell, could you please check this change is technically correct?

@jordandcarter
Copy link
Contributor

Hey @mbelton-buildkite, on behalf of @dabarrell I can say this change is technically correct 🙌

Copy link
Contributor

@mbelton-buildkite mbelton-buildkite left a comment

Choose a reason for hiding this comment

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

Thanks @jordandcarter! And thanks again, @nick-f. The linting has passed so I'll merge this for you. 😊

@mbelton-buildkite mbelton-buildkite merged commit 049e2d0 into buildkite:main Nov 13, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants