Skip to content

[splunk] Update the splunk logo to have transparent background#13655

Merged
kgeller merged 2 commits intoelastic:mainfrom
kgeller:splunk-logo-transparent
Apr 24, 2025
Merged

[splunk] Update the splunk logo to have transparent background#13655
kgeller merged 2 commits intoelastic:mainfrom
kgeller:splunk-logo-transparent

Conversation

@kgeller
Copy link
Contributor

@kgeller kgeller commented Apr 24, 2025

Proposed commit message

Updating the splunk logo svg to have a transparent background so it looks better in Kibana dark mode.

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

Related issues

Screenshots

Before:

image (2)

After:

Screenshot 2025-04-23 at 10 30 05 PM

(still looks good in light mode of course)
Screenshot 2025-04-23 at 10 39 12 PM

I downloaded the 'new' logo from Splunk's site here: https://www.splunk.com/en_us/form/sign-up.html

@kgeller kgeller added enhancement New feature or request Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Apr 24, 2025
@kgeller kgeller self-assigned this Apr 24, 2025
@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@kgeller kgeller marked this pull request as ready for review April 24, 2025 03:05
@kgeller kgeller requested a review from a team as a code owner April 24, 2025 03:05
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

Copy link
Contributor

@chemamartinez chemamartinez left a comment

Choose a reason for hiding this comment

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

LGTM, could you update the PR link in the changelog before merging it?

Co-authored-by: Chema Martínez <chema.martinez@elastic.co>
@kgeller kgeller enabled auto-merge (squash) April 24, 2025 11:46
@elastic-sonarqube
Copy link

@kgeller kgeller merged commit b8db6ef into elastic:main Apr 24, 2025
7 checks passed
@elasticmachine
Copy link

💚 Build Succeeded

History

cc @kgeller

@elastic-vault-github-plugin-prod

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Integration:splunk Splunk Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants