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

Cohere + AI monitoring -> LLM monitoring #10059

Merged
merged 3 commits into from
May 17, 2024
Merged

Cohere + AI monitoring -> LLM monitoring #10059

merged 3 commits into from
May 17, 2024

Conversation

colin-sentry
Copy link
Member

@colin-sentry colin-sentry commented May 16, 2024

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs
  • PR was reviewed and approved by a member of the Sentry docs team

Copy link

vercel bot commented May 16, 2024

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

Name Status Preview Comments Updated (UTC)
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 17, 2024 4:04pm

Copy link

codecov bot commented May 16, 2024

Bundle Report

Changes will increase total bundle size by 206 bytes ⬆️

Bundle name Size Change
sentry-docs-server 7.38MB 213 bytes ⬆️
sentry-docs-edge-server 458.69kB 3 bytes ⬇️
sentry-docs-client 6.16MB 4 bytes ⬇️

Copy link
Contributor

@vivianyentran vivianyentran left a comment

Choose a reason for hiding this comment

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

Added some copyedits but overall looks good!

Just one more thing — Can you make sure to add redirects for the product docs since their paths changed? Just in case anyone had the old links saved. See https://docs.sentry.io/contributing/pages/redirects/ for more info

docs/platforms/python/integrations/anthropic/index.mdx Outdated Show resolved Hide resolved
docs/platforms/python/integrations/openai/index.mdx Outdated Show resolved Hide resolved
docs/product/llm-monitoring/getting-started/index.mdx Outdated Show resolved Hide resolved
docs/product/llm-monitoring/getting-started/index.mdx Outdated Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

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

This screenshot still says "AI Monitoring". If the UI is also changing from AI to LLM, let's update the screenshots here too.

Copy link
Member Author

Choose a reason for hiding this comment

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

there will be a last round of screenshot updating before the launch in June!

Copy link
Contributor

Choose a reason for hiding this comment

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

This screenshot still says "AI Monitoring". If the UI is also changing from AI to LLM, let's update the screenshots here too.

docs/platforms/python/integrations/cohere/index.mdx Outdated Show resolved Hide resolved
docs/platforms/python/integrations/cohere/index.mdx Outdated Show resolved Hide resolved
docs/platforms/python/integrations/cohere/index.mdx Outdated Show resolved Hide resolved
Co-authored-by: vivianyentran <20403606+vivianyentran@users.noreply.github.com>
@colin-sentry colin-sentry enabled auto-merge (squash) May 17, 2024 15:47
@colin-sentry colin-sentry merged commit 28c0f08 into master May 17, 2024
6 checks passed
@colin-sentry colin-sentry deleted the cohere branch May 17, 2024 16:04
@github-actions github-actions bot locked and limited conversation to collaborators Jun 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants