Skip to content

[Cloud Security]Added new Google Cloud icon to be used on our component#163840

Merged
animehart merged 29 commits intoelastic:mainfrom
animehart:cis-gcp-cloud-shell-googleIcon
Aug 15, 2023
Merged

[Cloud Security]Added new Google Cloud icon to be used on our component#163840
animehart merged 29 commits intoelastic:mainfrom
animehart:cis-gcp-cloud-shell-googleIcon

Conversation

@animehart
Copy link
Copy Markdown
Contributor

Summary

Originally I made a PR to add the icon into eui Library, however they told me that they no longer support adding 3rd party logo into eui library as such we will need to add it to Kibana instead.
This Changes includes changing the GCP logo on

  • Findings flyout
  • Benchmark flyout
  • Dashboard
  • CSPM and CNVM onboarding page (GCP tab)
Screenshot 2023-08-14 at 9 47 25 AM Screenshot 2023-08-14 at 9 33 58 AM Screenshot 2023-08-14 at 9 35 36 AM Screenshot 2023-08-14 at 9 37 40 AM

@animehart animehart marked this pull request as ready for review August 14, 2023 16:51
@animehart animehart requested a review from a team as a code owner August 14, 2023 16:51
@animehart animehart added the Team:Cloud Security Cloud Security team related label Aug 14, 2023
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security)

@animehart animehart added v8.10.0 release_note:skip Skip the PR/issue when compiling release notes labels Aug 14, 2023
Copy link
Copy Markdown
Contributor

@opauloh opauloh left a comment

Choose a reason for hiding this comment

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

Looks great!

@animehart
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
cloudSecurityPosture 220 221 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
cloudSecurityPosture 255.1KB 258.2KB +3.1KB

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@animehart animehart merged commit 12be587 into elastic:main Aug 15, 2023
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label Aug 15, 2023
bryce-b pushed a commit that referenced this pull request Aug 22, 2023
…nt (#163840)

## Summary

Originally I made a PR to add the icon into eui Library, however they
told me that they no longer support adding 3rd party logo into eui
library as such we will need to add it to Kibana instead.
This Changes includes changing the GCP logo on 
- Findings flyout
- Benchmark flyout
- Dashboard
- CSPM and CNVM onboarding page (GCP tab)
<img width="795" alt="Screenshot 2023-08-14 at 9 47 25 AM"
src="https://github.com/elastic/kibana/assets/8703149/53e3aafa-7f3d-4c8f-b20c-ec365693ed88">

<img width="1223" alt="Screenshot 2023-08-14 at 9 33 58 AM"
src="https://github.com/elastic/kibana/assets/8703149/2fcdf9ad-8088-42c4-bb32-d6cdf12d02bc">
<img width="563" alt="Screenshot 2023-08-14 at 9 35 36 AM"
src="https://github.com/elastic/kibana/assets/8703149/5c63d03a-f929-4e5c-80e7-04d8327c7acb">
<img width="754" alt="Screenshot 2023-08-14 at 9 37 40 AM"
src="https://github.com/elastic/kibana/assets/8703149/cd5bc65b-3118-4d1c-a943-7a12518ad5e6">

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related v8.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants