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

packages/openai: Update logo and the documentation #13036

Merged
merged 1 commit into from
Mar 11, 2025

Conversation

shmsr
Copy link
Member

@shmsr shmsr commented Mar 10, 2025

Proposed commit message

Updated the OpenAI logo since OpenAI was rebranded in February and they now have a new logo. Also, updated the documentation to clearly explain that while we are collecting metrics that are stored in logs-*, although they are not logs. Many other integrations follow this practice, and to be consistent, we want to make sure the OpenAI package also does the same.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

@shmsr shmsr requested a review from a team as a code owner March 10, 2025 07:46
@shmsr shmsr self-assigned this Mar 10, 2025
@elasticmachine
Copy link

💚 Build Succeeded

cc @shmsr

Copy link
Contributor

@alaudazzi alaudazzi left a comment

Choose a reason for hiding this comment

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

LGTM

@shmsr shmsr merged commit 792494f into elastic:main Mar 11, 2025
7 checks passed
@elastic-vault-github-plugin-prod

Package openai - 1.1.0 containing this change is available at https://epr.elastic.co/package/openai/1.1.0/

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.

6 participants