Skip to content

fix none metadata - #7524

Closed
ChenghaoMou wants to merge 1 commit into
BerriAI:mainfrom
ChenghaoMou:fix/none-metadata
Closed

ChenghaoMou wants to merge 1 commit into
BerriAI:mainfrom
ChenghaoMou:fix/none-metadata

Conversation

@ChenghaoMou

Copy link
Copy Markdown

Fix None Medatadata

Relevant issues

Fixes #7522

Type

🐛 Bug Fix

Changes

  • Added or {} to avoid None values.

[REQUIRED] Testing - Attach a screenshot of any new tests passing locally

If UI changes, send a screenshot/GIF of working UI fixes

I tried to run the tests, but a lot of them failed because of missing api tokens. Not sure I should try to pass them for this small change.

@vercel

vercel Bot commented Jan 3, 2025 •

Copy link
Copy Markdown

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

Name Status Preview Comments Updated (UTC)
litellm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 3, 2025 4:29pm

@ishaan-jaff

Copy link
Copy Markdown
Contributor

fixed here @ChenghaoMou #7523

@ishaan-jaff

Copy link
Copy Markdown
Contributor

closing in favor of #7523

This branch was successfully deployed

1 active deployment
Preview — 08465887 Deployed Jan 3, 2025 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: None metadata not handled

2 participants