8000 fix: add log source to new bonsai logs by tyleroooo · Pull Request #1622 · dydxprotocol/v4-web · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: add log source to new bonsai logs #1622

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 15, 2025
Merged

fix: add log source to new bonsai logs #1622

merged 1 commit into from
Apr 15, 2025

Conversation

tyleroooo
Copy link
Contributor

It's hard to parse messages in datadog sadly

@tyleroooo tyleroooo requested a review from a team as a code owner April 15, 2025 00:41
@vercel Vercel
Copy link
vercel bot commented Apr 15, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
v4-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 0:44am
v4-testnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 15, 2025 0:44am

8000
@tyleroooo tyleroooo enabled auto-merge (squash) April 15, 2025 00:41
jaredvu
jaredvu approved these changes Apr 15, 2025
View reviewed changes
Copy link
Contributor
@jaredvu jaredvu left a comment

Choose a reason for hiding this comment

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

🚢

nit: the name of the function doesn't really hint at it being used for fetching, i think we can rename in the future to wrapFetchAndLogBonsaiError

@tyleroooo tyleroooo merged commit 346b161 into main Apr 15, 2025
9 checks passed
@tyleroooo tyleroooo deleted the tu/fix-logs-again branch April 15, 2025 00:45
tyleroooo added a commit that referenced this pull request Apr 15, 2025
@tyleroooo
fix: add log source to new bonsai logs (#1622)
bf84989
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jaredvu jaredvu jaredvu approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tyleroooo @jaredvu
0